-
Notifications
You must be signed in to change notification settings - Fork 7
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
Restyling event page & link to event list #89
Conversation
{ locale: 'it' } | ||
); | ||
|
||
const EventAction = ({ |
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.
in order to avoid defining this component on every renderer, could it be moved outside EventActions
as a standalone component (we could keep it in the same 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.
Great tip
[events] | ||
); | ||
const nextEvents = useMemo( | ||
() => sortEvents(filterComingEvents(events)), | ||
[events] | ||
); | ||
|
||
const hasMorePastEvents = events.length > MAX_PAST_EVENTS + nextEvents.length; |
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.
Maybe I didn't get the logic but I can't understand why nextEvents
should be involved in this condition
const hasMorePastEvents = events.length > MAX_PAST_EVENTS + nextEvents.length; | |
const hasMorePastEvents = filterPastEvents(events).length > MAX_PAST_EVENTS; |
Am I losing something?
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.
Indeed, my bad 👍
📚 Description
Closes #36
What
Why
The user didn't have a way to enter the event page but manually write the URL
🖼 Screenshots/Recordings
Desktop new experience
screen-capture.webm
Phone new experience
screen-capture.2.webm
📝 To Do