Skip to content

Chris/Get all user's collections upon login #379#483

Open
chris-nowicki wants to merge 14 commits intodevelopfrom
chris/get-all-user-collections
Open

Chris/Get all user's collections upon login #379#483
chris-nowicki wants to merge 14 commits intodevelopfrom
chris/get-all-user-collections

Conversation

@chris-nowicki
Copy link
Copy Markdown
Contributor

@chris-nowicki chris-nowicki commented Aug 27, 2024

fixes #379

Currently, we are getting the user's data separately on each page. We need to combine these into the initial call so we can cache the results.

All data is now fetched by the time /league/all route is loaded. I created a 2 new data store objects for storing leagues and entries.

I also updated the data store test and /league/all/page.tsx to account for the new cases to test.

  • User's data
  • Leagues the user belongs to
  • Entries per league the user has
  • Current week

@vercel
Copy link
Copy Markdown

vercel Bot commented Aug 27, 2024

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

Name Status Preview Comments Updated (UTC)
gridiron-survivor ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 17, 2024 3:14am
gridiron-survivor-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 17, 2024 3:14am

@appwrite
Copy link
Copy Markdown

appwrite Bot commented Aug 27, 2024

Gridiron Survivor Application 6616ea581ef9f5521c7d

Function ID Status Action
Your function has been successfully deployed.

Project name: Gridiron Survivor Application
Project ID: 6616ea581ef9f5521c7d

Function ID Status Action
userAuth 6626fef885a9f630442b ready Ready View Logs

Only deployments on the production branch are activated automatically. If you'd like to activate this deployment, navigate to your deployments. Learn more about Appwrite Function deployments.

💡 Did you know?
Appwrite has a Discord community with over 16 000 members. Come join us!

ryanfurrer
ryanfurrer previously approved these changes Aug 29, 2024
Copy link
Copy Markdown
Member

@ryanfurrer ryanfurrer left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@chris-nowicki
Copy link
Copy Markdown
Contributor Author

@shashilo just a reminder to review this PR.

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.

FEE: Get all user's collections upon login

2 participants