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

Support multiple-listed classes #18

Open
nathanmsmith opened this issue Nov 13, 2022 · 0 comments
Open

Support multiple-listed classes #18

nathanmsmith opened this issue Nov 13, 2022 · 0 comments

Comments

@nathanmsmith
Copy link
Member

The same course/section can be listed by multiple subject areas. Right now, Hotseat can't handle this: we consider each course to be a completely separate course. Here are some initial thoughts here:

Thoughts

DB schema:

  1. The multiple-listed status could change from term to term, but each row in our courses table refers to the same class across different quarters. As a result, we may want to maintain the many-to-many relationship of multiple-listing in a table separate from courses, with term as a constraint.

User-visible pages:

  1. We could try to combine multiple-listed classes into a single page, while keeping the current DB schema for courses and sections. But it can be hard to determine the "canonical" subject area for a class – which could actually be different based on the term and section.
  2. Alternatively, we could keep separate pages for multiple-listed classes (and keep the current DB schema), but provide links to the other multiple-listed classes and display reviews for the other equivalent courses on the current page.

Test cases

A most extreme example would be LIFESCI 192A:

image

More normal examples include COM SCI M51A / EC ENGR M16.

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

No branches or pull requests

1 participant