-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feature/ensembl endpoints and db model #7
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool to see a working example!
src/schug/endpoints/ensembl.py
Outdated
return ensembl_genes | ||
|
||
|
||
@router.get("/{ensembl_gene_id}", response_model=List[EnsemblTranscriptRead]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@router.get("/{ensembl_gene_id}", response_model=List[EnsemblTranscriptRead]) | |
@router.get("/{ensembl_gene_id}", response_model=List[EnsemblTranscript]) |
Right? How it was produced is not important to the caller
note to self: https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/204 use this status code because 404 (https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/404) means client requested resource doesn't exist when it does EDIT: seems more difficult than I thought because
and the return of my endpoints do have a body if wrong client input i.e. empty array Taking Henrik's http class from atlas |
This PR adds | fixes:
How to prepare for test:
ssh
to ...bash servers/resources/SERVER.scilifelab.se/update-[THIS_TOOL]-stage.sh [THIS-BRANCH-NAME]
How to test:
Expected outcome:
Review:
This version is a: