Skip to content

swagger files#2

Open
moondit wants to merge 5 commits intoslimandslam:mainfrom
moondit:feature/swagger
Open

swagger files#2
moondit wants to merge 5 commits intoslimandslam:mainfrom
moondit:feature/swagger

Conversation

@moondit
Copy link

@moondit moondit commented Jun 10, 2025

  • Introduces Swagger files for Market Data & Retail Trader APIs from Schwab Developer Portal

@slimandslam
Copy link
Owner

I definitely do not want to add axios as a dependency. Is there no way to re-work your PR using fetch instead of axios?

@moondit
Copy link
Author

moondit commented Jul 19, 2025

I definitely do not want to add axios as a dependency. Is there no way to re-work your PR using fetch instead of axios?

Hey @slimandslam - axios is not a requirement but it would definitely consolidate the process of checking the authorization token (example) to single spot through the use of interceptors instead of having it in every method defined in the SDK.

That said, here is an example of your existing implementation using the underlying SDK generated directly from the Swagger. By omitting the --axios flag when running the sdk command fetch is used.

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.

2 participants