Skip to content

[ODM-10883] Rework api documentation, switch to opeanapi-generator-gradle-plugin#62

Merged
genestack-okunitsyn merged 48 commits intodevelopfrom
feature/ODM-10883-api-documentation
Jan 11, 2024
Merged

[ODM-10883] Rework api documentation, switch to opeanapi-generator-gradle-plugin#62
genestack-okunitsyn merged 48 commits intodevelopfrom
feature/ODM-10883-api-documentation

Conversation

@genestack-okunitsyn
Copy link
Contributor

No description provided.

@genestack-okunitsyn genestack-okunitsyn force-pushed the feature/ODM-10883-api-documentation branch 4 times, most recently from ada1e93 to ca82a5a Compare December 18, 2023 12:00
@genestack-okunitsyn genestack-okunitsyn force-pushed the feature/ODM-10883-api-documentation branch from ca82a5a to 21a2ee6 Compare December 18, 2023 12:35
@genestack-okunitsyn genestack-okunitsyn changed the title Feature/odm 10883 api documentation [ODM-10883] Rework api documentation, switch to opeanapi-generator-gradle-plugin Dec 18, 2023
@genestack-okunitsyn genestack-okunitsyn force-pushed the feature/ODM-10883-api-documentation branch from 5a52959 to 0420794 Compare December 22, 2023 17:09
@genestack-okunitsyn genestack-okunitsyn force-pushed the feature/ODM-10883-api-documentation branch from 1be4d8d to e6aa021 Compare December 23, 2023 10:22
@genestack-okunitsyn genestack-okunitsyn force-pushed the feature/ODM-10883-api-documentation branch from f4423b1 to 540bed9 Compare December 23, 2023 15:26
@genestack-okunitsyn genestack-okunitsyn force-pushed the feature/ODM-10883-api-documentation branch from de4c77a to a9a6905 Compare December 26, 2023 15:11
@genestack-okunitsyn genestack-okunitsyn marked this pull request as ready for review January 2, 2024 18:12
@genestack-okunitsyn genestack-okunitsyn requested review from a team as code owners January 2, 2024 18:12
m-smazhevsky
m-smazhevsky previously approved these changes Jan 8, 2024
Copy link
Contributor

@m-smazhevsky m-smazhevsky left a comment

Choose a reason for hiding this comment

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

Looks good, I guess we need somebody from qa team to check the formatting in swagger

operationId: "getAllVariants"
produces:
- "application/json"
description: |+
Copy link
Contributor

Choose a reason for hiding this comment

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

what happened with summary?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I suppose it was removed in openapi specification V3. Couldn't find it in there.

Copy link
Contributor

Choose a reason for hiding this comment

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

shouldn't we replace it with title or smth?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We could try, I'll create one more subtask for this, to find would be it helpful or not, thank you.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Update, summary here.

@genestack-okunitsyn genestack-okunitsyn merged commit d34d967 into develop Jan 11, 2024
@genestack-okunitsyn genestack-okunitsyn deleted the feature/ODM-10883-api-documentation branch January 11, 2024 12:29
genestack-tg-gh-deploy added a commit that referenced this pull request Sep 16, 2024
…adle-plugin (#62)

* [ODM-10883] Fully reworked solution to openapi-generator

* [ODM-10883] Remove deprecated base path

* [ODM-10883] Fix expression atomic endpoint

* [ODM-10883] Try to fix test with openapi update

* [ODM-10883] Fix predefined headers in omics endpoints

* [ODM-10883] Add to all linkage

* [ODM-10883] Switch to openapi

* [ODM-10883] Make parameter nullable

* [ODM-10883] Application.json to search endpoint

* [ODM-10883] Set nullable with other approach

* [ODM-10883] Set nullable with other approach, doubled

* [ODM-10883] Bump gradle plugin

* [ODM-10883] Remove nullable

* [ODM-10883] Add additional properties as a string

* [ODM-10883] Add nullable

* [ODM-10883] Make feature string or dict

* [ODM-10883] JsonMetadata attributes should be oneOf object/string

* [ODM-10883] Test with 3.1.0 spec and null

* [ODM-10883] Make all specs 3.1.0

* [ODM-10883] Remove oneOf, make it same as default

* [ODM-10883] Try to fix it in new way

* [ODM-10883] Test remove properties from feature

* [ODM-10883] Set anyOf clearly

* [ODM-10883] Test add properties to object

* [ODM-10883] Add min properties 1

* [ODM-10883] Comment null for value

* [ODM-10883] Value directly as a number

* [ODM-10883] Remove readOnly from object

* [ODM-10883] Put values in other way

* [ODM-10883] VariantItem to properties

* [ODM-10883] Remove read-only

* Test another way

* Remove items: string

* [ODM-10883] Fix variantItem

* [ODM-10883] Remove readonly from Item entities

* [ODM-10883] Description readOnly

* [ODM-10883] Add genotype to variantItem

* [ODM-10883] Remove items to make it parse properly in lib

* [ODM-10883] Replace unused host to /

* [ODM-10883] Remove server section

* [ODM-10883] Remove swager annotation

* [ODM-10883] All requestBody as application/json

* [ODM-10883] Fix description for manageData

* [ODM-10883] All content-types */* as application/json

* [ODM-10883] Remove template from example

* Fix examples in jobs endpoint

* [ODM-10883] Fix examples

* [ODM-10883] Fix examples
tropnikovvl added a commit that referenced this pull request Apr 15, 2025
…adle-plugin (#62)

* [ODM-10883] Fully reworked solution to openapi-generator

* [ODM-10883] Remove deprecated base path

* [ODM-10883] Fix expression atomic endpoint

* [ODM-10883] Try to fix test with openapi update

* [ODM-10883] Fix predefined headers in omics endpoints

* [ODM-10883] Add to all linkage

* [ODM-10883] Switch to openapi

* [ODM-10883] Make parameter nullable

* [ODM-10883] Application.json to search endpoint

* [ODM-10883] Set nullable with other approach

* [ODM-10883] Set nullable with other approach, doubled

* [ODM-10883] Bump gradle plugin

* [ODM-10883] Remove nullable

* [ODM-10883] Add additional properties as a string

* [ODM-10883] Add nullable

* [ODM-10883] Make feature string or dict

* [ODM-10883] JsonMetadata attributes should be oneOf object/string

* [ODM-10883] Test with 3.1.0 spec and null

* [ODM-10883] Make all specs 3.1.0

* [ODM-10883] Remove oneOf, make it same as default

* [ODM-10883] Try to fix it in new way

* [ODM-10883] Test remove properties from feature

* [ODM-10883] Set anyOf clearly

* [ODM-10883] Test add properties to object

* [ODM-10883] Add min properties 1

* [ODM-10883] Comment null for value

* [ODM-10883] Value directly as a number

* [ODM-10883] Remove readOnly from object

* [ODM-10883] Put values in other way

* [ODM-10883] VariantItem to properties

* [ODM-10883] Remove read-only

* Test another way

* Remove items: string

* [ODM-10883] Fix variantItem

* [ODM-10883] Remove readonly from Item entities

* [ODM-10883] Description readOnly

* [ODM-10883] Add genotype to variantItem

* [ODM-10883] Remove items to make it parse properly in lib

* [ODM-10883] Replace unused host to /

* [ODM-10883] Remove server section

* [ODM-10883] Remove swager annotation

* [ODM-10883] All requestBody as application/json

* [ODM-10883] Fix description for manageData

* [ODM-10883] All content-types */* as application/json

* [ODM-10883] Remove template from example

* Fix examples in jobs endpoint

* [ODM-10883] Fix examples

* [ODM-10883] Fix examples
tropnikovvl added a commit that referenced this pull request Apr 15, 2025
…adle-plugin (#62)

* [ODM-10883] Fully reworked solution to openapi-generator

* [ODM-10883] Remove deprecated base path

* [ODM-10883] Fix expression atomic endpoint

* [ODM-10883] Try to fix test with openapi update

* [ODM-10883] Fix predefined headers in omics endpoints

* [ODM-10883] Add to all linkage

* [ODM-10883] Switch to openapi

* [ODM-10883] Make parameter nullable

* [ODM-10883] Application.json to search endpoint

* [ODM-10883] Set nullable with other approach

* [ODM-10883] Set nullable with other approach, doubled

* [ODM-10883] Bump gradle plugin

* [ODM-10883] Remove nullable

* [ODM-10883] Add additional properties as a string

* [ODM-10883] Add nullable

* [ODM-10883] Make feature string or dict

* [ODM-10883] JsonMetadata attributes should be oneOf object/string

* [ODM-10883] Test with 3.1.0 spec and null

* [ODM-10883] Make all specs 3.1.0

* [ODM-10883] Remove oneOf, make it same as default

* [ODM-10883] Try to fix it in new way

* [ODM-10883] Test remove properties from feature

* [ODM-10883] Set anyOf clearly

* [ODM-10883] Test add properties to object

* [ODM-10883] Add min properties 1

* [ODM-10883] Comment null for value

* [ODM-10883] Value directly as a number

* [ODM-10883] Remove readOnly from object

* [ODM-10883] Put values in other way

* [ODM-10883] VariantItem to properties

* [ODM-10883] Remove read-only

* Test another way

* Remove items: string

* [ODM-10883] Fix variantItem

* [ODM-10883] Remove readonly from Item entities

* [ODM-10883] Description readOnly

* [ODM-10883] Add genotype to variantItem

* [ODM-10883] Remove items to make it parse properly in lib

* [ODM-10883] Replace unused host to /

* [ODM-10883] Remove server section

* [ODM-10883] Remove swager annotation

* [ODM-10883] All requestBody as application/json

* [ODM-10883] Fix description for manageData

* [ODM-10883] All content-types */* as application/json

* [ODM-10883] Remove template from example

* Fix examples in jobs endpoint

* [ODM-10883] Fix examples

* [ODM-10883] Fix examples
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.

3 participants