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
Improve token auth migration #16822
Improve token auth migration #16822
Conversation
core/Updates/4.0.0-b1.php
Outdated
@@ -269,6 +246,9 @@ public function doUpdate(Updater $updater) | |||
|
|||
$updater->executeMigrations(__FILE__, $this->getMigrations($updater)); | |||
|
|||
// just in case it was not executed manually by a user... | |||
MigrateTokenAuths::migrate(); |
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.
Is this necessary? Since the migration is added above, if the user clicks 'Update', it will definitely run right? If so, this would be redundant. Or is there some way for the user to opt out of that specific migration running?
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.
true, it's not needed .will remove it now
Noticed the core:matomo4-migrate-token-auths command doesn't provide any output, maybe a 'Done.' message would be useful to have? |
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.
left one other comment, otherwise looks good
Updated it 👍 |
Description:
as discussed. will basically look like this:
Review