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 angularjs use from MobileMessaging #19410
Conversation
…angularjs-MobileMessaging
…omo-org/matomo into vue-remove-angularjs-MobileMessaging
…omo-org/matomo into vue-remove-angularjs-MobileMessaging
This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers |
This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers |
cb5dc01
to
f19c4ed
Compare
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.
Seems to work as expected. But will require matomo-org/plugin-CustomAlerts#121 to merged first
* remove use of angularjs from ScheduledReports plugin * remove extra space * built vue files * remove angularjs use from MobileMessaging, creating new AdminPage component for index.twig * built vue files * fix report binding * get propagation of property to child vue-entry components to work, if strangely * try removing prefix/suffix entirely from vue entry properties * revert back to suffixing an underscore * built vue files * add missing vue-entry property + add missing translations * update CustomAlerts submodule * remove duplicate translations * remove duplicate translation * remove use of unneeded html_attr * remove unneeded use of html_attr filter * update submodule Co-authored-by: sgiehl <stefan@matomo.org>
* remove use of angularjs from ScheduledReports plugin * remove extra space * built vue files * remove angularjs use from MobileMessaging, creating new AdminPage component for index.twig * built vue files * fix report binding * get propagation of property to child vue-entry components to work, if strangely * try removing prefix/suffix entirely from vue entry properties * revert back to suffixing an underscore * built vue files * add missing vue-entry property + add missing translations * update CustomAlerts submodule * remove duplicate translations * remove duplicate translation * remove use of unneeded html_attr * remove unneeded use of html_attr filter * update submodule Co-authored-by: sgiehl <stefan@matomo.org>
* remove use of angularjs from ScheduledReports plugin * remove extra space * built vue files * remove angularjs use from MobileMessaging, creating new AdminPage component for index.twig * built vue files * fix report binding * get propagation of property to child vue-entry components to work, if strangely * try removing prefix/suffix entirely from vue entry properties * revert back to suffixing an underscore * built vue files * add missing vue-entry property + add missing translations * update CustomAlerts submodule * remove duplicate translations * remove duplicate translation * remove use of unneeded html_attr * remove unneeded use of html_attr filter * update submodule Co-authored-by: sgiehl <stefan@matomo.org>
Description:
This PR is based off of #19402.
Changes:
Review