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 from CustomDimensions plugin #19486
Conversation
If you don't want this PR to be closed automatically in 28 days then you need to assign the label 'Do not close'. |
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 The code changes here are looking good. But I actually found two regressions:
- The site selector doesn't show the currently selected site
- The button to add a new dimensions doesn't seem to work anymore
…ng to respond to update:modelValue events
…ion for that site, go back to dimension list
@sgiehl should be fixed + one other issue I found |
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 now 👍
* remove angularjs from CustomDimensions plugin * apply after $location.url change * support use of SiteSelector directly in vue-entry when there is nothing to respond to update:modelValue events * if switching site during custom dimension edit and there is no dimension for that site, go back to dimension list * rebuild
Description:
Changes:
Review