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

Upgrade to Astro 4 #243

Merged
merged 31 commits into from
Aug 29, 2024
Merged

Upgrade to Astro 4 #243

merged 31 commits into from
Aug 29, 2024

Conversation

Southpaw1496
Copy link
Contributor

@Southpaw1496 Southpaw1496 commented Aug 28, 2024

This pull request updates the website from a base of Astro 1.6 to Astro 4.14. The website doesn't yet take advantage of modern Astro features like Content Collections or i18n routing, only the bare minimum has been done to get it working on modern versions of Astro.

Notable changes:


See preview on Cloudflare Pages: https://preview-243.quiltmc-org.pages.dev

The integration that provides the element no longer works in Astro 4.
- Updated all packages to their latest working versions
- Moved i18next-fluent-fs-backend to a quiltmc-owned repository.
- Moved to a Quilt fork of astro-i18next which rolls back to a working version and removes the default fs backend (we have a custom one)
We no longer have access to font-awesome pro, so we're moving to astro-icon
We're focusing on the most pressing changes (icon migration) for now. Content collection migration can come later.

This reverts commit 7f66e84.
The icon packs all had an older version.
Content Collections migration will come in a future update

This reverts commit 4fc4b97.
@Southpaw1496
Copy link
Contributor Author

Looking at Astro's v2 migration guide (they recommend following the migration guides in order), I think there's a couple of things that still need to happen, particularly around RSS. The v3 and v4 guides don't seem to apply to us, since the website is so out of date that they're mostly deprecating/removing features that we haven't had access to.

@Southpaw1496
Copy link
Contributor Author

This PR also happens to fix #119

@Southpaw1496 Southpaw1496 linked an issue Aug 29, 2024 that may be closed by this pull request
The problem was completely different to what I thought it would be, but it's good that I checked.
@Southpaw1496 Southpaw1496 merged commit 45472e8 into main Aug 29, 2024
2 checks passed
@Southpaw1496 Southpaw1496 deleted the astro-4-upgrade branch September 3, 2024 19:14
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.

Switch icon set Upgrade to Node 18/19, also upgrade Astro if possible
3 participants