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

Updated dining.py for current dining site #161

Merged
merged 2 commits into from
May 14, 2024

Conversation

akrishan11
Copy link
Contributor

  • Added functions to retrieve data from the Dine on Campus API used on the dining website
  • Did not include some methods included in old module for the sake of avoiding redundancy
  • Included functions get_locations(), get_location_hours(), and get_location_menu()
  • get_location_menu() is rather slow however this is due to making several requests to the API to ensure the location and period IDs used in the url are valid, as I noticed these changed for same location/period in the past year(hence assuming they are not static values)
  • Still need to write Unit Tests, but all functions were manually tested

@akrishan11 akrishan11 changed the title Updated dining module to retrieve Dine on Campus data Updated dining.py to retrieve Dine on Campus data May 5, 2024
@akrishan11 akrishan11 changed the title Updated dining.py to retrieve Dine on Campus data Updated dining.py for current dining site May 5, 2024
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

Successfully merging this pull request may close these issues.

None yet

2 participants