Skip to content

Port Global Mosquito Observations Dashboard to OL#430

Open
cblin321 wants to merge 13 commits into
open-lambda:mainfrom
cblin321:main
Open

Port Global Mosquito Observations Dashboard to OL#430
cblin321 wants to merge 13 commits into
open-lambda:mainfrom
cblin321:main

Conversation

@cblin321
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Member

@tylerharter tylerharter left a comment

Choose a reason for hiding this comment

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

Good work!

Comment thread docs/worker/apps.md Outdated
git clone https://github.com/UW-Madison-DSI/Global-Mosquito-Observations-Dashboard.git
```

Edit `docker-compose.yml`:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Instead of editing a compose file, how about you just give them a one-liner they can copy/paste to the terminal? One container, so just docker, no compose.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I think we can start one service by specifying a service name after doing compose up, still uses compose, but it's a one-liner that does what we want

Comment thread docs/worker/apps.md
Clone the repo:

```bash
git clone https://github.com/UW-Madison-DSI/Global-Mosquito-Observations-Dashboard.git
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is this necessary? The admin install can point directly to a git repo (as in the other example).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The problem with pointing directly to the repo is it's cloned into a temporary directory with os.MkdirTemp, so I thought it'd be better for UX (since we need to start the DB container) to clone the repo.

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