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
[Vue] remove use of angularjs from LanguagesManager plugin #19423
Conversation
…tomo-org/matomo into vue-remove-angularjs-LanguagesManager
This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers |
e221e52
to
3d8fe40
Compare
This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers |
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.
@diosmosis Seems vue was actually dropping the value for nonce
. Not sure why that is the case. Maybe nonce has some sort of special handling in vue 🤷
Renaming the variable to form-nonce fixed the problem though...
Yeah, noticed Vue does this, wasn't able to figure out why either. Might be related to https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/nonce |
* remove use of angularjs from LanguagesManager plugin * add translations * built vue files * get languages dropdown to work * remove use of unneeded html_attr escape * renamend variable Co-authored-by: sgiehl <stefan@matomo.org>
Description:
Changes:
Review