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.

Sign up for GitHub

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

Jump to bottom

Add an authorization command to ease direct requests to nextstrain.org's web API #229

Merged
merged 2 commits into from
Oct 25, 2022
Merged

Add an authorization command to ease direct requests to nextstrain.org's web API #229

merged 2 commits into from
Oct 25, 2022

Conversation

tsibley
Copy link
Member

@tsibley tsibley commented Oct 25, 2022 •
edited
Loading

  • Manually tested logged in and logged out cases and the example curl command in nextstrain authorization --help output
  • pytest passes locally
  • Checks pass

@tsibley tsibley requested a review from a team October 25, 2022 22:06
@tsibley tsibley merged commit 4149cb9 into master Oct 25, 2022
@tsibley tsibley deleted the trs/authorization-command branch October 25, 2022 22:48
victorlin
victorlin reviewed Oct 25, 2022
View reviewed changes
Copy link
Member

@victorlin victorlin left a comment

Choose a reason for hiding this comment

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

This is handy! @huddlej you can use this in nextstrain/seasonal-flu@f117ec0 once it's released.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@victorlin victorlin victorlin left review comments

Assignees
No one assigned
Labels
None yet
Projects
No open projects
Nextstrain planning (archived)
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tsibley @victorlin