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

#3144 Upgrade to Bootstrap 5 #3357

Closed
wants to merge 1 commit into from

Conversation

gjacob24
Copy link
Contributor

This commit contains the joint work of @johnpinto1 and @gjacob24 for the upgrade to Bootstrap 5 (#3144)

Version of code: v4.1.1

The CHANGELOG.md contains a record of all the major changes.

@github-actions
Copy link

github-actions bot commented Oct 30, 2023

2 Warnings
⚠️ There are code changes, but no corresponding tests. Please include tests if this PR introduces any modifications in behavior. \n
Ignore this warning if the PR ONLY contains translation.io synced updates.
⚠️ This PR is too big! Consider breaking it down into smaller PRs.

Generated by 🚫 Danger

@johnpinto1
Copy link
Contributor

johnpinto1 commented Nov 13, 2023

Please don't review PR yet. In the process of creating a wiki for this upgrade @gjacob24 and I decided to go through the following migration documents:

@gjacob24 had suggested we shouldn't rush PR creation before, advice I ignored wanting push code out because the issue was dragging on. Now I realise this was a bad idea, as to do a proper job we need to check code with above-mentioned documentation and remove obsolete classes, etc.

@johnpinto1 johnpinto1 force-pushed the issue_3144_upgrade_to_bootstrap_5 branch 3 times, most recently from 9eb9cd7 to 1bf4922 Compare November 29, 2023 10:32
@johnpinto1 johnpinto1 force-pushed the issue_3144_upgrade_to_bootstrap_5 branch 3 times, most recently from 26d9503 to 110ee1f Compare December 18, 2023 18:27
@johnpinto1 johnpinto1 force-pushed the issue_3144_upgrade_to_bootstrap_5 branch 5 times, most recently from c8c06ca to 38fd2ca Compare January 9, 2024 13:28
This commit consists of the file changes required for the upgrade from Bootstrap 3 to 5.  A more detailed version is in CHANGELOG.md (and in wiki: https://github.com/DMPRoadmap/roadmap/wiki/Release-notes-for-Bootstrap-5-upgrade)
@gjacob24 gjacob24 force-pushed the issue_3144_upgrade_to_bootstrap_5 branch from 611d64c to 932e31e Compare January 26, 2024 11:56
@gjacob24
Copy link
Contributor Author

Closing this PR to create another one.

@gjacob24 gjacob24 closed this Jan 26, 2024
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.

2 participants