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

begin backend migration #1

Merged
merged 3 commits into from
Apr 6, 2024
Merged

Conversation

GhostOf0days
Copy link
Member

need to migrate github actions in backend as well as deal with non-environment variable-related errors when running pnpm run build --filter backend after pnpm install at monorepo root

@GhostOf0days GhostOf0days changed the title begin migration begin backend migration Mar 16, 2024
@Adrastopoulos
Copy link
Member

Thanks for the effort. This is a good start. While we have the chance, we should take this opportunity to make some much-needed architectural changes. The main one is migrating from plain JSON api to TRPC, which this repo is set up for. This should be pretty simple to set up, just replace the controllers with routers.

We want to get rid of doppler and replace jest with vite. But this is a good start.

@Adrastopoulos
Copy link
Member

Made all the schemas. Couple of things needed:

  1. Extract all relations to a relations function, similar to the auth.ts schema
  2. Make all primary keys varchars instead of serial, similar to the auth.ts schema
  3. Add createdAt and updatedAt on all schemas.

@ScottyLabsDeploy ScottyLabsDeploy merged commit 6816257 into main Apr 6, 2024
2 of 3 checks passed
@ScottyLabsDeploy ScottyLabsDeploy deleted the Migrate-Backend-Into-Monorepo branch April 6, 2024 19:33
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.

3 participants