Skip to content
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

[4.0] pre-update check #33393

Merged
merged 1 commit into from
Apr 28, 2021
Merged

[4.0] pre-update check #33393

merged 1 commit into from
Apr 28, 2021

Conversation

brianteeman
Copy link
Contributor

when moving from bs2 to bs5 the label-* classes were changed to alert-* classes

however there is no such class as alert-important - it should be alert-danger

before

image

after

image

when moving from bs2 to bs5 the label-* classes were changed to alert-* classes

however there is no such class as alert-important - it should be alert-danger
@Quy
Copy link
Contributor

Quy commented Apr 28, 2021

I have tested this item ✅ successfully on f216014


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33393.

1 similar comment
@alikon
Copy link
Contributor

alikon commented Apr 28, 2021

I have tested this item ✅ successfully on f216014


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33393.

@alikon
Copy link
Contributor

alikon commented Apr 28, 2021

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33393.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Apr 28, 2021
@Quy Quy added PR-4.0-dev RTC This Pull Request is Ready To Commit and removed RTC This Pull Request is Ready To Commit labels Apr 28, 2021
@Quy Quy added this to the Joomla 4.0 milestone Apr 28, 2021
@chmst chmst merged commit a03a4d6 into joomla:4.0-dev Apr 28, 2021
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Apr 28, 2021
@chmst
Copy link
Contributor

chmst commented Apr 28, 2021

Thanks!

@brianteeman
Copy link
Contributor Author

Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants