Skip to content
This repository has been archived by the owner on Oct 18, 2024. It is now read-only.

feat: ✨ make single course/instructor query nullable (WIP) #116

Closed
wants to merge 1 commit into from

Conversation

ecxyzzy
Copy link
Member

@ecxyzzy ecxyzzy commented Dec 4, 2023

Creating PR as proof of concept for icssc/peterportal-client#351 (comment).

@ecxyzzy ecxyzzy temporarily deployed to staging-116-docs December 4, 2023 22:15 — with GitHub Actions Inactive
@ecxyzzy ecxyzzy temporarily deployed to staging-116 December 4, 2023 22:15 — with GitHub Actions Inactive
@ecxyzzy ecxyzzy temporarily deployed to staging-116 December 4, 2023 22:17 — with GitHub Actions Inactive
@ecxyzzy ecxyzzy temporarily deployed to staging-116-docs December 4, 2023 22:17 — with GitHub Actions Inactive
@ecxyzzy ecxyzzy temporarily deployed to staging-116 December 4, 2023 22:21 — with GitHub Actions Inactive
@ecxyzzy ecxyzzy temporarily deployed to staging-116-docs December 4, 2023 22:21 — with GitHub Actions Inactive
@ecxyzzy
Copy link
Member Author

ecxyzzy commented Dec 4, 2023

Closing since things aren't any faster. We probably need to rethink the representation of courses/instructors in the database.

@ecxyzzy ecxyzzy closed this Dec 4, 2023
@ecxyzzy ecxyzzy deleted the nullable-courses-instructors branch December 4, 2023 23:45
@ecxyzzy ecxyzzy restored the nullable-courses-instructors branch December 18, 2023 08:05
@ecxyzzy ecxyzzy deleted the nullable-courses-instructors branch December 18, 2023 08:08
ecxyzzy added a commit that referenced this pull request Dec 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant