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

🐱🐶Petstore Samples #993

Merged
merged 13 commits into from
Apr 4, 2023
Merged

🐱🐶Petstore Samples #993

merged 13 commits into from
Apr 4, 2023

Conversation

PureKrome
Copy link
Contributor

@PureKrome PureKrome commented Sep 12, 2022

Resolves Kiota-135 issue.

Currently, only the dotnet sample.

petstore/README.md Outdated Show resolved Hide resolved
petstore/README.md Outdated Show resolved Hide resolved
petstore/README.md Outdated Show resolved Hide resolved
petstore/README.md Outdated Show resolved Hide resolved
petstore/README.md Outdated Show resolved Hide resolved
petstore/README.md Outdated Show resolved Hide resolved
| -------- | ------------ |
| [dotnet](./dotnet) | `.\kiota.exe --openapi https://petstore.swagger.io/v2/swagger.json --additional-data false --language csharp -o PetstoreClient -c PetstoreApiClient --log-level information --clean-output true --namespace-name KiotaSamples.PetStoreSdk` |
| [java](./java) | |
| [typescript](./typescript) | |
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
| [typescript](./typescript) | |
| [typescript](./typescript) | `kiota generate --openapi https://petstore.swagger.io/v2/swagger.json --language typescript -o src -n petstore.utilities` |

@baywet baywet marked this pull request as ready for review April 4, 2023 19:14
@baywet baywet merged commit 6021dec into microsoft:main Apr 4, 2023
@PureKrome
Copy link
Contributor Author

Thanks heaps @baywet (and team) for merging this.

At various points I tried to get other lang versions (like TS and GO) but the learning curve (for me) was too much for the tiny amount of time I had to invest in those langs.

I even tried using ChatGPT to help get me through the majority of the conversion .. but it still didn't work.

So this brings me great joy, seeing as it's been handled!

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.

add pet store examples to the samples repo
2 participants