Merge and deploy latest #44
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sync to delphi
- deploy to dev paths - restrict access to known users - use dev database on localhost
make dev actually be dev
use dev branch badge
- 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
- grants access to the development vesion of the site from the private 127.16.0.0/12 subnet - used to allow unfettered access during local development in a docker environment - fix typo
- include table definitions, along with brief commentary - include a minimally functional set of development data - create the docker images needed for local development, along with documentation and required files - full local development guide to come in follow-up commit
enable local development
- database was updated with two new fields - use priority 0.9 to allow registration email to get through ahead of most other emails - use current timestamp
fix user registration
Troubleshoot signup issues
Mmm...not it
Is it this...
Comment out suspect function
- Describe updated development workflow (puts things roughly in line with our other more recent repos) - Remove deployment stats badging. We aren't deploying this in the old school way anymore so this is not useful or accurate to display any longer.
Co-authored-by: Katie Mazaitis <[email protected]>
Update Crowdcast for containerization and archive status
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.