Skip to content

importing relevant changes from dev #14

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

Merged
merged 8 commits into from
Apr 29, 2020
Merged

Conversation

dfarrow0
Copy link
Contributor

cherry-picks the parts of #13 that should also apply to the prod branch

- enables local development over http
- don't get the nowcast from the database as it's not used anyway
- the nowcast should be taken from the epidata api, rather than reading 
from the database directly
- include in header as this will be used across several pages in the 
near future
- remove outdated local copy of the library
- use hosted version as this will always be up-to-date with the epidata 
api
- remove the registration
- remove the worker's source code
- remove the deployment
- add code to unregister the worker, which is **persistent** in the
browser
- write some commentary on the situation
- removes final dependency on the epidata database, at least for 
"normal-season" epicast
- populate "latest issue" async in javascript in the places it's needed 
(it's only needed for dislpay purposes)
- update the list of required fields to reflect that some preferences 
are currently disabled
- fixes the bug where advanced preferences can't be updated
- css was interpreted as php in the past, but no longer
- removed the old css file
- removed the htaccess file which made this hack possible
- imports the ddl from the dev branch, which couldn't be cherry-picked 
because it was included alonside dev-only material
@dfarrow0
Copy link
Contributor Author

@undefx undefx merged commit 9a8dd80 into cmu-delphi:prod Apr 29, 2020
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.

2 participants