-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
UI: Ember deprecation - transition methods part 1 #25560
UI: Ember deprecation - transition methods part 1 #25560
Conversation
CI Results: |
}), | ||
|
||
actions: { | ||
navAfterSave({ saveType, model }) { |
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.
This method as a task was causing test failures
3308b92
to
ed3f374
Compare
global failure: TypeError: While generating link to route "vault.cluster.settings.auth.configure.section": Cannot read properties of undefined (reading 'shouldSupersede')
Global error: Uncaught TransitionAborted: TransitionAborted
Build Results: |
setupApplicationTest(hooks); | ||
|
||
hooks.beforeEach(async function () { | ||
await authPage.login(); | ||
return; | ||
}); | ||
|
||
skip('it allows create, list, delete of an entity alias', async function (assert) { | ||
test('it allows create, list, delete of an entity alias', async function (assert) { |
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.
🎉
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.
Great work!
This PR replaces
this.transitionTo
in routes andthis.transitionToRoute
in controllers everywhere except within the replication engine. That one is running into errors and is worth tackling in a separate PR.