-
Notifications
You must be signed in to change notification settings - Fork 12
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]: Auto update new courses in autocomplete graph #270
Comments
Note that a good long-term solution for this sort of issue will probably involve setting up a system to pull the latest DAG data from the API's |
Ah I didn't know the autocomplete endpoint was functional, I'll work on that |
On hold until the API endpoint is deployed |
Idk, might be a little misleading to display 3390 data for 2390 since there have probably been some class changes. Hopefully people would know to look up 3390, but the fact they have to do that would indicate it is different from 2390. |
This issue is blocked by nebula-api/#225
This is blocked by nebula-api/#223 |
When UTDNebula/nebula-api#223 is completed, it will in fact group ECS 2390 and ECS 3390. Now, we could detect the change maybe and let users know the course name has changed? That requires that the course API also groups by |
If API is doing the grouping, seems reasonable to just do that then |
Hi! I'm the one in charge of UTDNebula/nebula-api#223. The task will be completed by the end of Friday at the latest, and the issue with grouping will be resolved. |
We could probably look into making the grouping optional if such functionality would be preferred. |
Both of these are now unblocked via the latest API deployment :) |
What did you expect to happen?
The ability to search for new courses from the current (24f) semester, i.e. ECS 2390.
What happened?
The course cannot be searched and isn't provided via autocomplete despite the course data being present in the API.
Version
Beta 0.1.0
What browsers are you seeing the problem on?
Firefox
Relevant log output
No response
Code of Conduct
The text was updated successfully, but these errors were encountered: