-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
Event Location and related models #23
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@fikrialwan and @babdurrahman78 can use the same existing branch to work together, no need to create a separated branch. |
@mhaidarhanif this pull request is ready now, for review together, I possible tonight |
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.
- Fix import to use
~
such as~/components
- Remove organizerId requirement in model admin event
- Fix layout to edit the detail to be simpler in the middle
- Event Category radio button background color
- Event page to move Category to below Format, with just a text
- Rename Location (Offline) to Location (In Person)
- Show Maps URL below Location In Person
- Fix render if location is null
- Don't show Format and Category label if the data is unavailable
@mhaidarhanif
|
What kind of change does this PR introduce?
Create function and UI to add event location when create event.
Event Location is available if the Category is In Person.
What is the current behavior?
What is the new behavior?
new behaviour event page
Closes #5