Skip to content

convert to a static site #361

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

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from
Draft

convert to a static site #361

wants to merge 8 commits into from

Conversation

afeld
Copy link
Contributor

@afeld afeld commented Feb 8, 2025

As I was looking through the code for the site, it seemed to me that the application is more complicated than it needs to be. I took a crack at rebuilding it with 11ty, a static site generator for NodeJS.

I tried keeping the changes as minimal as possible, though there is a lot that can be deleted if y'all are ok with this direction.

This also has implications for the hosting. I suggest to switching to something like Netlify as an alternative.

Happy to follow through with this — let me know if I should.

TODOs

For parity with the routes:

  • / - confirmed the map worked with switch to MapLibre #360
  • /about
  • /search
  • /add
  • /contribute
  • /catalog/:id - will need to set up redirects with the hosting provider
  • /portal/:id
  • /group/:id
  • /api/data.json
  • /api/catalogs/:id
  • /api/catalogs
  • /api/groups
  • /admin/reload - no longer relevant

@afeld afeld marked this pull request as draft February 8, 2025 06:23
@afeld afeld force-pushed the 11ty branch 2 times, most recently from 62449f5 to 79ea457 Compare April 7, 2025 01:49
@afeld
Copy link
Contributor Author

afeld commented Apr 7, 2025

@todrobbins Whatcha think?

@todrobbins
Copy link
Member

todrobbins commented Apr 7, 2025

@afeld makes total sense to me. MapLibre is a great choice and there was a ton of old node/npm baggage that I'm glad you've cleaned out. I need to build locally to test myself, but I love it!

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