-
-
Notifications
You must be signed in to change notification settings - Fork 98
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: merge spa-scratch branch #133
feat: merge spa-scratch branch #133
Conversation
Co-authored-by: Jonas Lagoni <jonas-lt@live.dk>
Co-authored-by: Maciej Urbańczyk <urbanczyk.maciej.95@gmail.com> Co-authored-by: David Boyne <davidboyne@Davids-MBP.lan>
Kudos, SonarCloud Quality Gate passed!
|
@fmvilas We have some small bugs on sonarcloud but I think that they are not so necessary to fix. In the next PR I will update Readme as well as I will configure semantic-release. Please approve :) |
Happy with the branch, 215 files are quite big to review, but the application and it's architecture seems to be in a working state when I was working with it (looked good). So based on my experience I'm happy with it going in. Guess any details we can review/refactor as we come across them? |
🚀 |
Description
Merge
spa-scratch
branch to themaster
:asyncapi-react
componentRelated issue(s)
See #80