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

feat: Support basePath in options-per-service for openapi generator #5311

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

KavithaSiva
Copy link
Contributor

@KavithaSiva KavithaSiva commented Dec 23, 2024

Cherry picked commit for PR - #5247

  • I know which base branch I chose for this PR, as the default branch is v3-main now, which is not for v4 development.
    - [ ] If my change will be merged into the main branch (for v4), I've updated (V4-Upgrade-Guide.md)[./V4-Upgrade-Guide.md] in case my change has any implications for users updating to SDK v4

… - Option 2 (#5247)

* chore: append basePath to path pattern in openapi generator

* chore: switch to using removeSlashes from util

* chore: add additional test

* chore: check-public-api change

* Changes from lint:fix

* chore: update test

* chore: adjust all tests

* chore: fix codeQL warning

* Changes from lint:fix

* chore: add changeset

* chore: cleanup tests

* feat: extend openapi request builder for basePath

* chore: introduce setBasePath

* feat: introduce _defaultBasePath to the generated files

* chore: fix order of parameters with intro of basepath

* chore: fix tests

* chore: add setBasePath

* chore: fix failing e2e test

* Changes from lint:fix

* chore: cleanup path

* Changes from lint:fix

* chore: adjust test

* Changes from lint:fix

* chore: switch to using undefined for default basePath value

* chore: cleanup test

* chore: update changeset

* chore: adjust changeset

* chore: fix unit test

* chore:cleanup

* chore: fix test

* chore: cleanup changesets

* fix: integration test

* chore: fix path

* chore:fix path

* chore: remove dead code

* chore: review comment

* Update .changeset/chilled-flowers-hide.md

Co-authored-by: Deeksha Sinha <[email protected]>

* Update packages/openapi/src/openapi-request-builder.ts

Co-authored-by: Deeksha Sinha <[email protected]>

* chore: address review comments

* chore: revert update to removeSlashes utility

* Changes from lint:fix

* chore: address review comments

* chore: tests fix

* chore: adjust tests

* chore: address review comments

* Changes from lint:fix

* fix: begin all paths using '/'

* chore: address review comments

* Update .changeset/chilled-flowers-hide.md

Co-authored-by: Deeksha Sinha <[email protected]>

* Update .changeset/chilled-flowers-freeze.md

Co-authored-by: Deeksha Sinha <[email protected]>

* Update packages/util/src/remove-slashes.spec.ts

Co-authored-by: Deeksha Sinha <[email protected]>

* Update packages/util/src/remove-slashes.spec.ts

Co-authored-by: Deeksha Sinha <[email protected]>

* chore: address review comments

* Changes from lint:fix

* chore: add review changes

* chore: address review comments

* chore:address review comments

* chore: address review comments

* chore: release note fix

---------

Co-authored-by: cloud-sdk-js <[email protected]>
Co-authored-by: Junjie Tang <[email protected]>
Co-authored-by: Deeksha Sinha <[email protected]>
@KavithaSiva KavithaSiva changed the title feat: Support basePath in options-per-service for openapi generator… feat: Support basePath in options-per-service for openapi generator Dec 23, 2024
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.

1 participant