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

Refactor and fix types in event pages #4973

Closed
wants to merge 14 commits into from
Closed

Refactor and fix types in event pages #4973

wants to merge 14 commits into from

Conversation

eikhr
Copy link
Member

@eikhr eikhr commented Sep 22, 2024

!!! Appearantly this PR was "too large" and had "too many changes to review", so I will split it up a bit. The first part is here #4974

Description

While working on ABA-1070 I got a bit annoyed at how much spaghetti and type errors there was. So i did some refactoring:)

Result

If you've made visual changes, please check the boxes below and include images showing the changes. Descriptions are appreciated.

  • 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.

No visual changes.

Fixes the loading indicator when unregistering people in the admin list.

Testing

  • I have thoroughly tested my changes.

Please describe what and how the changes have been tested, and provide instructions to reproduce if necessary.

Copy link

vercel bot commented Sep 22, 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) Visit Preview Sep 22, 2024 11:19am

@github-actions github-actions bot added the review-needed Pull requests that need review label Sep 22, 2024
@eikhr eikhr marked this pull request as draft September 22, 2024 11:34
@eikhr eikhr closed this Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review-needed Pull requests that need review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant