-
Notifications
You must be signed in to change notification settings - Fork 0
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
14 events summary #45
Conversation
Heyo! Code looks good but I can't test right now - I'm getting a merge error when I try to pull, was this branch rebased and safe to force pull? |
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.
Initialising LGTM but strapi configurability desirable - will raise this during meeting :3
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.
Looks good! Eventually it'll be good to get the event description in Strapi but this is a nice initialisation
Yep I defo think we should add Strapi to this |
I'll merge this in now since it's been approved - we can add Strapi in a later PR! |
Context
Adding a hardcoded title and paragraph to the events summary page to provide users with an introductory overview of the events
Also added authorisation in strapi.ts
Closes #14
What Changed?
How To Review
Testing
Tested the events summary page make sure the title and paragraph display correctly
Notes