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

Migrate to Postgres #497

Merged
merged 59 commits into from
Nov 6, 2024
Merged

Migrate to Postgres #497

merged 59 commits into from
Nov 6, 2024

Conversation

js0mmer
Copy link
Member

@js0mmer js0mmer commented Nov 1, 2024

Description

  • Migrated to postgres
  • added drizzle
  • refactored routes
    • made sure we are determining userid from session, not letting user dictate it
  • refactored login checks on the frontend with a hook
  • added course bag routes
  • added separate course bag slice

todo

  • need to clean up mongoose stuff before merging
  • update docs

Steps to verify/test this change:

  • Verify changes work as expected on staging instance
  • make sure the site still works

Final Checks:

  • Verify successful deployment

(optional)

  • Write tests
  • Write documentation

Issues

Closes #490

@js0mmer js0mmer temporarily deployed to staging-497 November 1, 2024 22:46 — with GitHub Actions Inactive
@js0mmer js0mmer temporarily deployed to staging-497 November 4, 2024 02:25 — with GitHub Actions Inactive
@js0mmer js0mmer temporarily deployed to staging-497 November 4, 2024 02:27 — with GitHub Actions Inactive
@js0mmer js0mmer temporarily deployed to staging-497 November 4, 2024 09:07 — with GitHub Actions Inactive
@js0mmer js0mmer temporarily deployed to staging-497 November 4, 2024 09:17 — with GitHub Actions Inactive
@js0mmer js0mmer temporarily deployed to staging-497 November 4, 2024 21:59 — with GitHub Actions Inactive
@js0mmer js0mmer temporarily deployed to staging-497 November 4, 2024 23:56 — with GitHub Actions Inactive
Copy link
Member

@Awesome-E Awesome-E left a comment

Choose a reason for hiding this comment

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

Just create the issues for the two unresolved comments, otherwise looks pretty much good to me! Though it wouldn't hurt to get a second opinion from @timobraz

@js0mmer js0mmer temporarily deployed to staging-497 November 6, 2024 00:39 — with GitHub Actions Inactive
@js0mmer js0mmer temporarily deployed to staging-497 November 6, 2024 00:39 — with GitHub Actions Inactive
@js0mmer js0mmer temporarily deployed to staging-497 November 6, 2024 00:42 — with GitHub Actions Inactive
@js0mmer js0mmer temporarily deployed to staging-497 November 6, 2024 07:55 — with GitHub Actions Inactive
@js0mmer js0mmer temporarily deployed to staging-497 November 6, 2024 20:12 — with GitHub Actions Inactive
@js0mmer js0mmer temporarily deployed to staging-497 November 6, 2024 20:20 — with GitHub Actions Inactive
@js0mmer js0mmer merged commit 1b2b063 into master Nov 6, 2024
2 checks passed
@js0mmer js0mmer deleted the drizzle branch November 6, 2024 20:23
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.

Migrate to Postgres
2 participants