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

fixing requestParameters schema for genomicVariations #47

Closed
wants to merge 2 commits into from

Conversation

mbaudis
Copy link
Member

@mbaudis mbaudis commented Aug 11, 2022

This seems like the correct structure but needs to be filled in for the correct references (filters etc.)

While this fixes the wrong g_variants root parameter the PR is mostly the basis for discussion & expansion.

This then also needs to be transferred to the JSON version, after editing is finished.

This seems like the correct structure but needs to be filled in for the correct references (filters etc.)
@mbaudis mbaudis requested a review from redmitry August 11, 2022 16:11
... other parameters then to be added later.
@mbaudis mbaudis changed the title prototyping requestParameters schema for genomicVariations fixing requestParameters schema for genomicVariations Aug 12, 2022
@mbaudis
Copy link
Member Author

mbaudis commented Aug 12, 2022

IMO should be merged in as fix of the wrongly formatted schema. w/o need to hold back.

@mbaudis mbaudis requested a review from tb143 August 12, 2022 07:50
mbaudis added a commit that referenced this pull request Aug 15, 2022
In the Beacon v2 default model, the variant request parameters for the query documents (i.e. POST queries) are wrapped in the `g_variant` object. While this ATM is incorrectly formatted (see proposed fix in #47) the intention is now visible here in the query documents for v2 POST examples.
@mbaudis mbaudis added this to the 2.1 milestone Dec 20, 2022
@costero-e costero-e closed this Mar 28, 2023
@costero-e costero-e deleted the genomicVariations-requestParameters-fix branch March 28, 2023 11:48
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