-
Notifications
You must be signed in to change notification settings - Fork 4
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
Update token version #498
Update token version #498
Conversation
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.
hey @yeniatencio
Thanks for the changes.
I think we should not limit the version to a specific old version, especially when the latest version has passed the Drupal security tests. Please keep it at ^1.11 and update the patch.
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.
noice! @yeniatencio !
Jira
https://digital-vic.atlassian.net/browse/SD-134
Problem/Motivation
Patch doesn't apply in the last version of token as the code has been refactored. Therefore, downgrading to version 12 and update to latest available version of the patch.
Fix
https://www.drupal.org/project/token/issues/2924873#comment-15705267
Related PRs
Screenshots
TODO