Skip to content

Aud/fea/toast notifications#189

Merged
AudreyBil merged 7 commits intomainfrom
aud/fea/toast-notifications
Mar 16, 2026
Merged

Aud/fea/toast notifications#189
AudreyBil merged 7 commits intomainfrom
aud/fea/toast-notifications

Conversation

@AudreyBil
Copy link
Copy Markdown
Collaborator

This small PR adds new toast notifications where it makes sense + remove all console logs, warnings, errors.

Toast notifications added:

  • event created
  • event saved as draft
  • event creation failed
  • location created
  • Location creation failed
  • File removal
  • Account creation

Fix console warnings:
On homepage:

  • Root index route with no Component: React router needs something in Component for leaf routes. I added Component: () => null as the loader always redirects before anything renders.
  • No HydrateFallback element provided to rend during initial hydration: Irrelevant for us as we do not do SSR, I used HydrateFallback:() => null to silence the warning.
  • A few other react warning

Other Fix:
Only public events where showing on the Event Feed page. Now, public events + privates events the user is going to or is invited too are also showing up in the feed.

Console errors left:
Favicon

@AudreyBil AudreyBil marked this pull request as ready for review March 16, 2026 14:38
@AudreyBil AudreyBil merged commit 9df5475 into main Mar 16, 2026
4 checks passed
@AudreyBil AudreyBil deleted the aud/fea/toast-notifications branch March 16, 2026 19:10
@AudreyBil AudreyBil mentioned this pull request Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants