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.

Sign up for GitHub

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

Jump to bottom

Move record page to nextjs #20

Merged

timhabermaas merged 4 commits into master from move-record-page-to-nextjs

Jan 18, 2022

Owner

timhabermaas commented Dec 17, 2021 •

edited

Loading

Changes route from /users/[slug]/records/[id] to just /records[id] in
the process.

Todo

Handle 404s gracefully.
Redirect old URLs to new URL at nginx level (have been shared using Facebook).

timhabermaas added 4 commits

December 17, 2021 21:03


          Move record page to nextjs

4d746e0

Changes route from /users/[slug]/records/[id] to just /records[id] in
the process.


          Format times with penalties on record page correctly

05ee95a


          Display 404 error if some resource doesn't exist

e0c1f8c


          Move <Layout> to _app.tsx

2c00807

timhabermaas merged commit c16c3ec into master

timhabermaas deleted the move-record-page-to-nextjs branch

January 18, 2022 21:33

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

Labels

None yet