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

revert course catalog changes, fix oauth callback #594

Merged
merged 6 commits into from
Dec 20, 2024

Conversation

PThorpe92
Copy link
Member

@PThorpe92 PThorpe92 commented Dec 17, 2024

This renames all the improper spelling of catalog, and reverts the changes to UserCourses that were made because of a seeder XY problem.

Also fixes the issue where users are not automatically logged into canvas upon clicking on a course

https://gist.github.com/PThorpe92/c693588b16452a6cfb488cc71a91a27f
Schema changes documented here

backend/src/database/activity.go Outdated Show resolved Hide resolved
backend/src/database/user_catalog.go Outdated Show resolved Hide resolved
@PThorpe92 PThorpe92 marked this pull request as draft December 18, 2024 17:20
@PThorpe92 PThorpe92 added WIP refactor - backend HAS MIGRATION This PR has a migration added. Please collaborate with the author to resolve the ordering labels Dec 18, 2024
@PThorpe92 PThorpe92 marked this pull request as ready for review December 20, 2024 03:47
@PThorpe92 PThorpe92 merged commit 59d0a45 into UnlockedLabs:main Dec 20, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
HAS MIGRATION This PR has a migration added. Please collaborate with the author to resolve the ordering refactor - backend WIP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants