-
-
Notifications
You must be signed in to change notification settings - Fork 6.8k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
[typescript-node] Promise-based Authentication interface #3408
Conversation
👍 Thanks for opening this issue! The team will review the labels and make any necessary changes. |
cc @TiFu (2017/07) @taxpon (2017/07) @sebastianhaas (2017/07) @kenisteward (2017/07) @Vrolijkx (2017/09) @macjohnny (2018/01) @nicokoenig (2018/09) @topce (2018/10) @akehir (2019/07) |
👋 @wing328 |
modules/openapi-generator/src/main/resources/typescript-node/api-single.mustache
Show resolved
Hide resolved
modules/openapi-generator/src/main/resources/typescript-node/api-single.mustache
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the quick review!
modules/openapi-generator/src/main/resources/typescript-node/api-single.mustache
Show resolved
Hide resolved
modules/openapi-generator/src/main/resources/typescript-node/api-single.mustache
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks @macjohnny ! Will this go out with the next v4.X.X minor release (v4.1.0, I think)? |
Yes, this should be part of 4.1.0 |
The main change here is that APIs' authorization is now handled asynchronously. See OpenAPITools/openapi-generator#3408 for details.
@silasbw thanks for the PR, which has been included in the 4.1.0 release: https://twitter.com/oas_generator/status/1160000504455319553 |
PR checklist
./bin/
to update Petstore sample so that CIs can verify the change. (For instance, only need to run./bin/{LANG}-petstore.sh
,./bin/openapi3/{LANG}-petstore.sh
if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates). Windows batch files can be found in.\bin\windows\
. If contributing template-only or documentation-only changes which will change sample output, be sure to build the project first.master
,4.1.x
,5.0.x
. Default:master
.Description of the PR
Example implementation for #3407.
fixes #3407