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

Clean up remaining Bootstrap 3 code #501

Merged
merged 1 commit into from
Oct 28, 2024
Merged

Conversation

sea-kelp
Copy link
Collaborator

@sea-kelp sea-kelp commented Sep 9, 2024

Description of Changes

Cherry-picked from lucyparsons#1123

Migrates off bootstrap3 and into bootstrap5. Thank you @psjalltheway !!!

Removes deprecated files which are causing conflicts for the new HTML/CSS which will come in a separate PR.

Notes for Deployment

None, although we might need to clean up the unused nginx asset files?

Screenshots (if appropriate)

N/A

Testing instructions

I ran just fresh-start locally and made sure styles were still rendered correctly

Checks

  • I have rebased my changes on main

  • just lint passes

  • just test passes

Migrates off bootstrap3 and into bootstrap5. Thank you @psjalltheway !!!

Removes deprecated files which are causing conflicts for the new
HTML/CSS which will come in a separate PR.

- [x] This branch is up-to-date with the `develop` branch.
- [x] `pytest` passes on my local development environment.
- [x] `pre-commit` passes on my local development environment.

---------

Co-authored-by: psjalltheway <[email protected]>
Co-authored-by: sea-kelp <[email protected]>
@sea-kelp sea-kelp requested a review from a team as a code owner September 9, 2024 05:43
Comment on lines +32 to +43
def strtobool(val: str) -> bool:
# Implemented based on distutils strtobool documentation
# https://docs.python.org/3.8/distutils/apiref.html#distutils.util.strtobool
val = str(val).lower()

if val in ["y", "yes", "t", "true", "on", "1"]:
return True

if val in ["n", "no", "f", "false", "off", "0"]:
return False

raise ValueError(f"string cannot be converted to boolean: {val}")
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added our own strtobool function since I was getting a build error which I think was caused by distutils being deprecated

Copy link
Collaborator

@AetherUnbound AetherUnbound left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is amazing! 🤩 Everything runs fine locally, so glad that we're able to tear out so much 🚀

@AetherUnbound AetherUnbound merged commit 7ca2af6 into main Oct 28, 2024
2 checks passed
@AetherUnbound AetherUnbound deleted the bootstrap3-deprecation branch October 28, 2024 22:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants