-
Notifications
You must be signed in to change notification settings - Fork 5
Added links for different events #17
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
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @BigSpaceships ! Tysm for making these changes! Happy to see someone take on this service.
Overall looks good! Made some comments around some small typing asks.
Also, can you provide screenshots or video of the UI on desktop and mobile to demonstrate the changes and confirm it works on both?
I did notice that you can get a random json response if you go to a deleted event so Ima fix that too |
Added links for when you're going between open events or events in the history page (makes the back button work and fixes #15)
Also added share link button to event details which copies permalink to the event