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

Production - Close Up Map #15

Open
wants to merge 42 commits into
base: production
Choose a base branch
from

Conversation

cheese-cracker
Copy link

Corresponding MASTER branch pull request has to be merged. This for the Close Up MAP.

Rahul Bothra and others added 30 commits July 22, 2018 11:01
* Update media files
* Update warden numbers
* Update timings
* menu json files path was missing a '/' at the end

Tested-by: MananSoni42
* Media files were moved to bob-the-bot repository from pro-panda.github.io
    * Remove developers names and redundant info
* Add MIT license to src/menu_update.py

* Add food menu files for Aug 2018
* modularize
* remove db from master in favor of production
* Update media files
* Update warden numbers
* Update timings
* Add media files
* menu json files path was missing a '/' at the end

Tested-by: MananSoni42
    * Remove developers names and redundant info
only works locally when running test_app.py

changes for feature:
* changed get_meal function
* added entries to commondict.txt, mapping.json

changes for local testing:
* added temporary menu files
* query_sort.py (get_name)
* changed db/ to ../db/ in spell_correct, query_sort
…into add_next_day_menu

remove local testing code
* Add feature: tomorrow's food menu is shown
only works locally when running test_app.py

changes for feature:
* changed get_meal function
* added entries to commondict.txt, mapping.json

changes for local testing:
* added temporary menu files
* query_sort.py (get_name)
* changed db/ to ../db/ in spell_correct, query_sort
@rhl-bthr rhl-bthr force-pushed the production branch 3 times, most recently from 7447c61 to f4f2516 Compare October 29, 2019 19:47
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.

3 participants