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

[Bug] No default text for missing fields in CourseCard #205

Open
ap-1 opened this issue Feb 2, 2025 · 1 comment
Open

[Bug] No default text for missing fields in CourseCard #205

ap-1 opened this issue Feb 2, 2025 · 1 comment
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@ap-1
Copy link
Contributor

ap-1 commented Feb 2, 2025

Expected Behavior

This should ideally say "No description available" and "Unknown units" or something to that extent, instead of just rendering nothing.

Image

Steps to Reproduce

See 21-469 as an example

Device Information

All

@ap-1 ap-1 added bug Something isn't working good first issue Good for newcomers labels Feb 2, 2025
@xavilien
Copy link
Member

xavilien commented Feb 3, 2025

This is a good point, but in general there shouldn't be a reason for there to be empty descriptions or units. I looked through the database and it seems like for some reason there's about 77 courses with empty descriptions and units (and I think the coreqs and prereqs got emptied out as well). E.g. 21235 and 21237 which definitely do have descriptions and units on the schedule of classes. I'm not entirely sure why that is the case but it could be a case of the course scraper not doing its job well or the querying of data from the CMU Schedule of Classes going awry. Course scraper does run every night so we can check back tonight to see if this is fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants