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 Nextjs14 and Amplify #170

Merged
merged 16 commits into from
Jun 16, 2024
Merged

Upgrade to Nextjs14 and Amplify #170

merged 16 commits into from
Jun 16, 2024

Conversation

skorfmann
Copy link
Contributor

@skorfmann skorfmann commented Jun 14, 2024

This switches to Nextjs 14 (app router) and Amplify as a backend with the goal to use Amplify Hosting.

The main change is, that the data won't be coming from Github anymore, but from the backend directly (DDB). Links can be suggested via a public (😱) form. Will receive an email notifications for new links via SNS. Speaking of SNS, dropped the dead newsletter form and replaced it with a simple SNS subscription for new posts.

once merged, the /frontend part can be removed and /website can be moved to /

@skorfmann
Copy link
Contributor Author

here's a preview https://nextjs14.d2vgttdm5hfz44.amplifyapp.com/ /cc @mbonig - still needs some polishing for the forms and potentially some caching optimization. But essentially it's good to go. The github auth can be added after merge as well

@skorfmann
Copy link
Contributor Author

also, disabled the netlify integration

@skorfmann skorfmann marked this pull request as ready for review June 14, 2024 15:57
@skorfmann
Copy link
Contributor Author

FYI: Gonna merge this today as is. Will expand with streamlining the link adding afterwards.

@skorfmann skorfmann merged commit 205de85 into master Jun 16, 2024
1 check passed
@skorfmann skorfmann deleted the nextjs14 branch June 16, 2024 20:05
This was referenced Jun 16, 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.

1 participant