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

Migrate front tRPC to api routes #52

Merged
merged 27 commits into from
May 6, 2024
Merged

Migrate front tRPC to api routes #52

merged 27 commits into from
May 6, 2024

Conversation

hubcio2115
Copy link
Owner

@hubcio2115 hubcio2115 commented Apr 19, 2024 •
edited
Loading

Closes #49
Closes #51

Changes

majors:

  • moved entire api from trpc to next api routes
  • added @total-typescript/ts-reset
  • added eslint-plugin-drizzle
  • added an error handler when creating a new organization and the desired name has been already taken
  • added auth middleware for /dashboard/*

minors:

  • updated nextjs to 14.2.2

fix:

  • there was a bug where after adding a new org the list of orgs in the select wouldn't update

@hubcio2115 hubcio2115 added the enhancement New feature or request label Apr 19, 2024
@hubcio2115 hubcio2115 added this to the Final milestone Apr 19, 2024
@hubcio2115 hubcio2115 self-assigned this Apr 19, 2024
@vercel Vercel
Copy link

vercel bot commented Apr 19, 2024 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
editthing ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 1, 2024 9:34am

@hubcio2115 hubcio2115 marked this pull request as ready for review April 19, 2024 17:59
@hubcio2115 hubcio2115 changed the title updated packages Migrate front tRPC to api routes Apr 19, 2024
@hubcio2115 hubcio2115 removed the request for review from Ciszkoo May 1, 2024 09:34
kalisowski
kalisowski approved these changes May 6, 2024
View reviewed changes
Copy link
Collaborator

@kalisowski kalisowski left a comment

Choose a reason for hiding this comment

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

Nice work
image

@kalisowski kalisowski merged commit e32944f into main May 6, 2024
3 checks passed
@hubcio2115 hubcio2115 deleted the feat/migrate-from-trpc-to-api-routes branch May 7, 2024 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kalisowski kalisowski kalisowski approved these changes

@Maxxxxxxxxxxxxxxxxxxxxx Maxxxxxxxxxxxxxxxxxxxxx Maxxxxxxxxxxxxxxxxxxxxx approved these changes

Assignees

@hubcio2115 hubcio2115

Labels
enhancement New feature or request
Projects
None yet
Milestone
Final
Development

Successfully merging this pull request may close these issues.

Stworzyć middleware Zmigrować aplikację z TRPC na Next API Routes
3 participants
@hubcio2115 @kalisowski @Maxxxxxxxxxxxxxxxxxxxxx