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] migrate remaining directives in CorePluginsAdmin #18433
Conversation
…#18497) * forward compareSegments in when applying period selector change + check if comparing periods in period selector * built vue files
…class to use kebab case, check for field using css class as well in Widget.vue
…to vue-plugin-settings
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.
probably wait to if there is any other comments
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.
Code overall looks fine. Did a quick testing of the UI and it seems everything works fine.
The JS tests are failing, but that seems to be caused by the last PR already. Might be good to fix them before merging.
UI test failures seem to be unrelated to this PR.
@@ -25,15 +25,9 @@ | |||
|
|||
$timeout(function () { |
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.
Any reason why this file is still needed?
Description:
This PR is based off of #18432.
Changes:
Changes made to make UI tests pass:
fetch()
polyfill library: https://github.com/github/fetch. See the AbortController spec: https://developer.mozilla.org/en-US/docs/Web/API/AbortControllerReview