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 SegmentEditor plugin #19391
Conversation
b4a24a8
to
ed04407
Compare
…o-org/matomo into vue-remove-angularjs-SegmentEditor
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 |
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 Only had a quick look at the UI yet, but it seems when switching the segment in the selector, the selector doesn't change it's value / highlight. Even though the segment seems to be loaded correctly.
…o-org/matomo into vue-remove-angularjs-SegmentEditor
@sgiehl fixed the issue you mentioned. Did some more manual testing, but not a comprehensive test, and couldn't find anything else, but the Segmentation.js code can be confusing so it's very possible I missed something. |
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.
Looks good to me now. Clicked through the UI and didn't find any further regressions.
The segment editor in custom reports won't work though until the PR to remove angular was merged there as well.
* remove use of angularjs from SegmentEditor plugin * remove import added by phpstorm * make sure to export comparison service instance not just class * built vue files * fixing some issues * fix initial value for segment definition * treat null, undefined and empty string segment value the same * Update expected screenshots * try to fix timing error in test * remove use of html_attr escape * adds missing translations * remove unused code * fix segment definition selection and fix Matomo URL watch in Segmentation.js * fix test failure Co-authored-by: sgiehl <stefan@matomo.org>
Description:
Changes:
Review