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

feat: update open content, library, and video fields in tables/queries/frontend #588

Merged
merged 9 commits into from
Dec 17, 2024

Conversation

calisio
Copy link
Contributor

@calisio calisio commented Dec 13, 2024

Description of the change

Renames columns on open content providers, library, video fields for consistency and updates wherever they were used. Reduces open content item structs into one. Also fixes visibility status query for favorites and frontend for that.

@calisio calisio added the HAS MIGRATION This PR has a migration added. Please collaborate with the author to resolve the ordering label Dec 13, 2024
@calisio calisio self-assigned this Dec 13, 2024
Copy link
Contributor

@carddev81 carddev81 left a comment

Choose a reason for hiding this comment

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

backend/models/dashboard.go (line 23)
change 'name' to 'title' - unable to save a new helpful link due to column 'name' not existing

backend/migrations/00027_alter_open_content_tables.sql Outdated Show resolved Hide resolved
backend/src/database/DB.go Show resolved Hide resolved
@PThorpe92 PThorpe92 merged commit 9f81377 into UnlockedLabs:main Dec 17, 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants