Skip to content

fix: redirect /event-page to Google Form#275

Merged
need4deed merged 1 commit into
developfrom
fix/event-page-redirect
Apr 22, 2026
Merged

fix: redirect /event-page to Google Form#275
need4deed merged 1 commit into
developfrom
fix/event-page-redirect

Conversation

@nadavosa
Copy link
Copy Markdown
Collaborator

Summary

  • need4deed.org/event-page was showing "no information" because the events API returns nothing
  • Replaces the EventPage component with an immediate redirect to the Google Form

Related

Closes need4deed-org/fe#312

Test plan

  • Visit need4deed.org/event-page and confirm it redirects to the Google Form

🤖 Generated with Claude Code

…nt list

Closes need4deed-org/fe#312

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@nadavosa nadavosa requested a review from need4deed April 22, 2026 12:09
@nadavosa nadavosa enabled auto-merge April 22, 2026 12:11
@nadavosa nadavosa disabled auto-merge April 22, 2026 12:12
@Cy-fox
Copy link
Copy Markdown

Cy-fox commented Apr 22, 2026

Hey @nadavosa, looks good overall, the redirect approach is correct.
Just a few things to clean up, since you're redirecting immediately, these are all unused now: Lang, useTranslation, useEvents (it's still making an API call for no reason), Event, StaticPageLayout. The commented-out return can go too.

@nadavosa
Copy link
Copy Markdown
Collaborator Author

Thanks @Cy-fox! The file already has all those removed — only useEffect remains with the redirect and return null. No unused imports, no API calls.

@need4deed need4deed merged commit a2447ff into develop Apr 22, 2026
@need4deed need4deed deleted the fix/event-page-redirect branch April 22, 2026 13:14
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.

Redirect url of event page

3 participants