-
-
Notifications
You must be signed in to change notification settings - Fork 113
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
633 event post type #1140
base: master
Are you sure you want to change the base?
633 event post type #1140
Conversation
WalkthroughThe recent updates introduce an 'EVENT' post type across the application, enabling users to create, update, and manage events with specific details like date and location. These changes span multiple files, from backend schemas and GraphQL type definitions to frontend components, ensuring a cohesive functionality for handling event data. Changes
Assessment against linked issues
The current updates effectively lay down the groundwork for handling event-type posts with necessary attributes and operations. However, the extension to a communal calendar view and its interaction needs further development or clarification. Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
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.
Actionable comments posted: 1
At a quick glance, this creates the
Do you want to create the calendar view for it? If yes, we can pop this into draft. If no, I can review but it won't be merged until someone does a calendar view for it. |
Yeah I can do a calendar view no prob. Should it be a calendar view with all the events or just the single event? |
It should have all the events. If I'm in |
Description
closes #633
Screenshots
Additional Context
New migration for enum was added
Checklist
Are your changes backwards compatible? Please answer below:
I believe so
Did you QA this? Could we deploy this straight to production? Please answer below:
I did QA this. Seemed to work correctly, but I added a lot of things. From my knowledge
For frontend changes: Tested on mobile? Please answer below:
Yes
Did you introduce any new environment variables? If so, call them out explicitly here:
No