-
Notifications
You must be signed in to change notification settings - Fork 7
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
Module still includes a check for the token module. #6
Comments
Token is now part of core and always enabled + hidden I think we can safely remove that check. |
jenlampton
pushed a commit
to jenlampton/colorbox
that referenced
this issue
May 22, 2017
jenlampton
added a commit
that referenced
this issue
May 22, 2017
Issue #6: Remove check for legacy token module.
There's a section related to this that can still be cut out: https://github.com/backdrop-contrib/colorbox/blob/1.x-2.x/colorbox.module#L402-L413 |
laryn
added a commit
to laryn/colorbox
that referenced
this issue
Feb 3, 2022
laryn
added a commit
to laryn/colorbox
that referenced
this issue
Feb 3, 2022
laryn
added a commit
to laryn/colorbox
that referenced
this issue
Feb 3, 2022
Sorry, I had some unrelated changes that kept getting included in there but have a clean PR on this one now. |
jenlampton
added a commit
that referenced
this issue
Feb 3, 2022
Issue #6: Remove notice about installing token module.
Thanks @laryn, PR merged! |
jenlampton
pushed a commit
that referenced
this issue
Apr 11, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
At colorbox.module line 390 there is a check that at line 409 produces a warning in Backdrop.
The text was updated successfully, but these errors were encountered: