-
Notifications
You must be signed in to change notification settings - Fork 113
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
fix: fix all mutated props #335
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lots of i18n strings have been changed, which is not right.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have you tested each ones of the view with our Cypress tests to see if something is broken? I don’t think it would fail, but still.
100% Cypress tests OK! |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just checked the code itself - I haven't run the tests or Cypress to see if it still works.
Good job!
One thing to validate though - see comment.
🎉 This PR is included in version 0.1.0-alpha.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This pull request has been automatically locked since there |
Remove all mutated props in Vue pages.
You fool, don't forget these steps: