Skip to content
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

Include registration statuses to event items #5230

Conversation

ivarnakken
Copy link
Member

@ivarnakken ivarnakken commented Dec 12, 2024

Description

Makes it much easier to differentiate between the list of dates in the items.
There's no need to show the activation time for registrations if the user has
already registered.

Besides, people prefer to use the event list on /events rather than the
one on the user profile to view upcoming events.

Result

  • Changes look good on both light and dark theme.
  • Changes look good with different viewports (mobile, tablet, etc.).
  • Changes look good with slower Internet connections.

Look at Deloitte AS and Fadderpåmelding Kommunikasjonsteknologi to see event items with 2 dates ... difficult to differentiate between the two:

Before After
image image

Testing

  • I have thoroughly tested my changes.

Tested all event types, and the three different registration statuses (unregistered, waiting list and registered/admitted)

`userReg` is now included in the read event `ListEvent` model
Makes it easier to differentiate between the list of dates in the items.
Besides, people prefer to use the event list on /events rather than the
one on the user profile to view upcoming events.
In case the user is on the /events page and logs in, the events should
be re-fetched to get the user registration statuses.
@ivarnakken ivarnakken added enhancement Pull requests that make enhancements, instead of just purely new features review-needed Pull requests that need review new-feature Pull requests that introduce a new feature technical-debt Pull requests that reduces technical debt labels Dec 12, 2024
@ivarnakken ivarnakken requested a review from a team December 12, 2024 14:08
@ivarnakken ivarnakken self-assigned this Dec 12, 2024
Copy link

linear bot commented Dec 12, 2024

Copy link

vercel bot commented Dec 12, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
lego-bricks-storybook ⬜️ Ignored (Inspect) Dec 12, 2024 2:08pm

Copy link
Member

@norbye norbye left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love it!! Have been missing a nice overview of events I've registered to for so long

app/components/EventItem/index.tsx Show resolved Hide resolved
@ivarnakken ivarnakken added the approved Pull requests that have been approved label Dec 12, 2024
@ivarnakken ivarnakken merged commit b5b403b into master Dec 17, 2024
8 checks passed
@ivarnakken ivarnakken deleted the ivarnakken/aba-1196-differentiate-between-dates-on-compact-events-better branch December 17, 2024 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Pull requests that have been approved enhancement Pull requests that make enhancements, instead of just purely new features new-feature Pull requests that introduce a new feature review-needed Pull requests that need review technical-debt Pull requests that reduces technical debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants