Skip to content

refactor: use Supabase for events and projects data#163

Merged
ehrelevant merged 8 commits into
up-csi:mainfrom
syncopascual:eventsAndProjects
May 14, 2026
Merged

refactor: use Supabase for events and projects data#163
ehrelevant merged 8 commits into
up-csi:mainfrom
syncopascual:eventsAndProjects

Conversation

@syncopascual

@syncopascual syncopascual commented May 10, 2026

Copy link
Copy Markdown
Contributor

Closes issue #160 by fetching data for events and projects from Supabase.

Implementation Notes

  • Edited models to match with the Supabase schema
  • Edited index.ts files for projects and events
  • Supabase

Misc. Notes

  • When navigating to /people, /projects, and /events there's a slight rendering delay that happens prolly due to the data being fetched from Supabase? Instead of navigating to the page first and then the page being populated with cards as the data is fetched, the page takes a bit to load for a split second, which might feel bad especially on a worse internet connection.

@ehrelevant ehrelevant left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!! Though yeah, we will have to investigate what can be done about the initial load times, since my guess is as good as yours at the moment.

P.S: Don't forget to set up commit signing.

@ehrelevant

Copy link
Copy Markdown
Contributor

Actually, wait, you mentioned "Instead of navigating to the page first and then the page being populated with cards as the data is fetched, the page takes a bit to load for a split second..." On my end, the page load is very fast, but it takes a while for the cards to load (which is somewhat expected, albeit could be improved). Was this what you were talking about, or was it something else?

@ehrelevant ehrelevant merged commit fc7a852 into up-csi:main May 14, 2026
1 check passed
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.

2 participants