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.

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

Account system (part) #14

Merged
merged 54 commits into from
Jan 18, 2024
Merged

Account system (part) #14

merged 54 commits into from
Jan 18, 2024

Conversation

jsun969
Copy link
Member

@jsun969 jsun969 commented Jan 18, 2024

Account system branch is too large. Merge it first and divide #9 todos to smaller issues and PRs.

CleanShot 2024-01-18 at 20 32 17@2x

phoenixpereira and others added 30 commits December 28, 2023 15:07
@rayokamoto
Copy link
Contributor

Yeah 51 commits is a bit crazy, ideally some of these commits should be squashed before it is merged. Or we can use the "squash and merge" strategy.

.env.local.example Outdated Show resolved Hide resolved
src/app/not-found.tsx Outdated Show resolved Hide resolved
src/components/UserButton.tsx Outdated Show resolved Hide resolved
src/middleware.ts Show resolved Hide resolved
@phoenixpereira phoenixpereira added frontend backend priority:high High priority; should be worked on immediately labels Jan 18, 2024
@rayokamoto rayokamoto merged commit d037200 into main Jan 18, 2024
3 checks passed
@rayokamoto rayokamoto mentioned this pull request Jan 18, 2024
19 tasks
@phoenixpereira phoenixpereira deleted the account-system branch January 18, 2024 11:17
@rayokamoto rayokamoto removed the priority:high High priority; should be worked on immediately label Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants