Victor Lin

Results 123 issues of Victor Lin

I just followed the tutorial, and it failed on Unit 4/5: **Create an Azure Static Web App**. The failure is visible on the automatically generated workflow run [here](https://github.com/victorlin/my-static-web-app/actions/runs/3843963412). It can...

### Current Behavior I thought I had the Conda runtime set up on my computer so I used `nextstrain shell --conda`. It resulted in a process that hung indefinitely using...

bug

Nextstrain CLI is currently provided in our managed runtimes, but this "CLI-ception" comes with some odd behavior. It's only noticeable in a few commands related to runtime management (`setup`, `update`)...

bug

### Current Behavior ```sh # Start local nextstrain.org server # Log in to the testing Cognito user pool export NEXTSTRAIN_DOT_ORG=http://localhost:5000 export NEXTSTRAIN_COGNITO_USER_POOL_ID="$(jq -r .COGNITO_USER_POOL_ID ../nextstrain.org/env/testing/config.json)" export NEXTSTRAIN_COGNITO_CLI_CLIENT_ID="$(jq -r .OAUTH2_CLI_CLIENT_ID ../nextstrain.org/env/testing/config.json)"...

bug

Currently, dependencies are defined rather loosely: https://github.com/nextstrain/ncov-ingest/blob/e6c904f1e36c6f833ee0b1b5b52b0aaa8570b6f8/requirements.txt#L1-L8 From https://github.com/nextstrain/ncov-ingest/pull/416#discussion_r1298885003: > Ideally, it would still be nice to lock these deps so they're not changing and liable to break on new...

enhancement

First noticed in https://github.com/nextstrain/nextstrain.org/pull/811#discussion_r1571704216. I didn't get the server log in time, but just noticed it again in [another CI run](https://github.com/nextstrain/nextstrain.org/actions/runs/8789741033/job/24120164067?pr=835). This is the test failure: ``` ● smoke testing...

bug

Recently, I've noticed that promoting canary to production prevents nextstrain.org from loading for a short but noticeable amount of time. With the latest promotion of 24ba9ee742e14c30a09ea03d4db3f8d6acefdc40 ([nextstrain-server](https://dashboard.heroku.com/apps/nextstrain-server) v894 → v895),...

bug

## Description of proposed changes The previous "links" relied on onClick which is bad for accessibility. If div is still desired, the link role would be appropriate.¹ Or, use an...

@jameshadfield [said](https://github.com/nextstrain/nextstrain.org/pull/763#issuecomment-1846410403): > Looking at the cards on /pathogens excluding the West-African EBOV dataset, only 6/11 are being kept up to date. So given this state of the page as...