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

fix: remove ejs jquery and bootstrap requirements for homepage #638

Merged
merged 2 commits into from
Dec 27, 2024

Conversation

brittonhayes
Copy link
Member

What does this do?

  • simplified the frontend setup by moving from EJS templates to a single static index.html
  • replaced the jQuery-based JavaScript with native fetch API calls
  • removed bootstrap css and got the same style with our own very minimal css

How was it tested?

  1. Verifying the home page loads correctly with the new static HTML
  2. Testing that the interactive API endpoint works properly with the native fetch API
  3. Testing across different screen sizes to ensure responsive design works
  4. Ensuring all CSS changes maintain the existing visual design

Is there a Github issue this is resolving?

No specific GitHub issue is being resolved. This is a proactive maintenance simplification effort.

Was any impacted documentation updated to reflect this change?

No documentation updates were needed since these changes are purely frontend-focused and don't impact any API functionality or documentation.

Result

The site looks exactly the same and works the same, just no more *.ejs, bootstrap, or jquery required to serve it. Just plain simple html and css.
image

Here's a fun image for your troubles

random photo - update me

Copy link
Collaborator

@bagelbits bagelbits left a comment

Choose a reason for hiding this comment

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

Thank you!

@bagelbits bagelbits merged commit ed1ff38 into main Dec 27, 2024
9 checks passed
@bagelbits bagelbits deleted the fix/remove-ejs-jquery-and-bootstrap branch December 27, 2024 02:25
Copy link

🎉 This PR is included in version 3.10.5 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

2 participants