Skip to content

feat: add typescript-axios template#83

Merged
mnahkies merged 10 commits into
mainfrom
mn/feat/typescript-axios
Dec 2, 2023
Merged

feat: add typescript-axios template#83
mnahkies merged 10 commits into
mainfrom
mn/feat/typescript-axios

Conversation

@mnahkies
Copy link
Copy Markdown
Owner

@mnahkies mnahkies commented Oct 1, 2023

  • new template typescript-axios based on https://github.com/axios/axios

  • compared to the typescript-fetch template it's quite "lossy", eg: you don't retain as much information about possible error responses. as we can't type hint possible exceptions I don't see a way to avoid this (short of catching and returning a result type, but this would seem to defeat the purpose of using axios)

  • included a couple of scripts using the github api as a test / sample

  • Documentation

  • Publish Alpha

Some dog fooding has been done, but not as much as the fetch template yet. Likely to some rough edges that will need to be fixed forward.

Testing
Alpha version published as:

@nahkies/openapi-code-generator@0.1.1-alpha.13
@nahkies/typescript-axios-runtime@0.1.1-alpha.13

@mnahkies mnahkies force-pushed the mn/feat/typescript-axios branch 2 times, most recently from 6c905e2 to 4493236 Compare November 5, 2023 11:41
Comment thread integration-tests/typescript-axios/package.json Outdated
@mnahkies mnahkies force-pushed the mn/feat/typescript-axios branch from 4493236 to 0662375 Compare November 5, 2023 11:44
@mnahkies mnahkies force-pushed the mn/feat/typescript-axios branch from f6a8b43 to 8cab6cd Compare November 29, 2023 19:09
@mnahkies mnahkies force-pushed the mn/feat/typescript-axios branch from 97848ab to 2ceb67d Compare December 2, 2023 09:11
@mnahkies mnahkies force-pushed the mn/feat/typescript-axios branch from 2ceb67d to 88ff2b1 Compare December 2, 2023 09:24
@mnahkies mnahkies marked this pull request as ready for review December 2, 2023 09:26
@mnahkies mnahkies merged commit c3e045d into main Dec 2, 2023
@mnahkies mnahkies deleted the mn/feat/typescript-axios branch December 2, 2023 09:27
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