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

First readme update #18

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

First readme update #18

wants to merge 1 commit into from

Conversation

sebastianbarry
Copy link

docker/load_mongodump.sh

  • Added the load_mongodump.sh file from e-mission/em-public-dashboard

README.md

  • Reordered the Docker method to go at the top of the How to run this app section
  • Added a "How to load data into the app" section

docker/load_mongodump.sh
- Added the load_mongodump.sh file from e-mission/em-public-dashboard

README.md
- Reordered the Docker method to go at the top of the How to run this app section
- Added a "How to load data into the app" section
@sebastianbarry
Copy link
Author

@shankari @swastis10 adding you both for visibility

`docker run dash-app`
1. Run the app (steps shown above)
2. Copy your `.tar.gz` data file into this directory (`/op-admin-dashboard/<name of data file>.tar.gz`). If you do not have this file, speak with @shankari to get access to these files
3. Run the following shell script: `bash viz_scripts/docker/load_mongodump.sh vail_2023-01-01.tar-2.gz`. This will load the data into the admin dashboard
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sebastianbarry this should not be a hardcoded file name, right?
Also, for the record, you don't need to copy the file into this directory. You can load it from anywhere as long as you specify the full file path. That's Unix CLI 101.

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