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

Rest API V2.0 - Technical Testing Feedback #75

Open
4 of 11 tasks
sprintell opened this issue Jan 27, 2025 · 0 comments
Open
4 of 11 tasks

Rest API V2.0 - Technical Testing Feedback #75

sprintell opened this issue Jan 27, 2025 · 0 comments
Assignees

Comments

@sprintell
Copy link
Member

sprintell commented Jan 27, 2025

  • Problem of All endpoints produces JSON containing embedded collection names as classNameList format.

  • This should be fixed by customizing the collection names using @relation and @collectionRelation annotation on on all the Data Transfer Objects

  • Problem of all the endpoints renders _links before payload content.

  • This should be fixed by specifying the order using the @JsonPropertyOrder on all the Data Transfer Objects

Screenshot 2025-01-27 at 10.21.22.png

  • Rest API keys should be consistent all through, some are in camelCase while some are in snake case, we should probably just maintain snake case all through

  • URLs pattern should be consistent as well, some are in kebab case while some are in camel case, we should probably just maintain kebab case all through

  • A link should exist between SNPs and GenomicController

  • Authors should not be embedded in publication be a separate endpoint linked to Publications

  • /gwas/rest/api/v2/associations/163449091/loci endpoint has some mapping error hence not displaying standard JSON payload

  • url parameters must be snake case not camelcase - efoTrait in association endpoint and other places

  • Global Exception handlers are not added. exception yet to be properly handled across all endpoints e.g http://gwas-snoopy:8087/gwas/rest/api/v2/studies/GCST90297764678

@sprintell sprintell self-assigned this Jan 27, 2025
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

No branches or pull requests

1 participant