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.

Sign up for GitHub

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

Jump to bottom

Add Server Action + Next 13.5 #27

Merged
merged 6 commits into from
Oct 4, 2023
Merged

Add Server Action + Next 13.5 #27

merged 6 commits into from
Oct 4, 2023

Conversation

Lucieo
Copy link
Contributor

@Lucieo Lucieo commented Sep 21, 2023 •
edited
Loading

Issue
#8

Server actions sur

  • delete invitation
  • update team info
  • update user

@vercel
Copy link

vercel bot commented Sep 21, 2023 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
digestclub ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 4, 2023 10:00am

@Lucieo Lucieo changed the title Cache issue Add sever action Sep 21, 2023
@Lucieo Lucieo changed the title Add sever action Add Server Action Sep 22, 2023
@Lucieo Lucieo marked this pull request as ready for review September 22, 2023 13:41
@Lucieo Lucieo changed the title Add Server Action Add Server Action + Next 13.5 Sep 25, 2023
@Lucieo Lucieo requested a review from baptadn September 25, 2023 08:51
@baptadn
Copy link
Member

baptadn commented Oct 2, 2023

Little conflicts 🐞

@Lucieo
Copy link
Contributor Author

Lucieo commented Oct 2, 2023

Fixed conflicts

@Lucieo
Copy link
Contributor Author

Lucieo commented Oct 3, 2023

Conflicts fixed

@Lucieo
Copy link
Contributor Author

Lucieo commented Oct 4, 2023

⚠️ we have an issue with unstable cache use when updating team info with next predicted digest name cache of checkUserTeamBySlug is not updated. We cannot use revalidateTag here as it's not a server action

@Lucieo
Copy link
Contributor Author

Lucieo commented Oct 4, 2023 •
edited
Loading

No more cache on checkUserTeamBySlug

@baptadn baptadn merged commit 093f677 into main Oct 4, 2023
3 checks passed
@baptadn baptadn deleted the feat/server-actions branch October 4, 2023 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@baptadn baptadn Awaiting requested review from baptadn

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Lucieo @baptadn