Bootstrap 5 migration #6307
Conversation
1b2efd3 to
680d08e
Compare
|
@sagargg It's so great to see progress here. How are you getting on with this? Do you need any support from the tech team or other devs? |
1b51bc5 to
27dd57b
Compare
4483060 to
cf21905
Compare
2689b72 to
de2ffa7
Compare
|
Hi @pdelboca @amercader, What is blocking here to merge this PR? |
910be03 to
f7db7cc
Compare
|
@sagargg it's mostly time constraints, I'll intend to have some final review of this hopefully next week. |
|
We discussed this on the dev meeting and once the bs3 templates use bs3 javascript files, this can be merged, rest of the issues can be fixed later on as the next ckan release is not scheduled yet. |
|
@Zharktas could you double check please. bs3 resources are loading for me. |
|
@Zharktas fixed broken UI in admin trash page. |
|
Bah, there where two different settings, which I don't like very much but that's how it was previously in similar migration so no need to change that. Can you update the error messages to contain valid options here and here and add a changelog entry in |
|
@Zharktas Done. |
Zharktas
left a comment
There was a problem hiding this comment.
Just a minor thing to fix and then I'll merge this :)
Co-authored-by: Jari Voutilainen <[email protected]>
Co-authored-by: Jari Voutilainen <[email protected]>


Fixes #
#6012
Proposed fixes:
Bootstrap upgrade 5
Features:
Please [X] all the boxes above that apply