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

[Feature Request]: mix this.configuration and _options in api method #58

Closed
1 task done
NotEvenANeko opened this issue Apr 28, 2023 · 2 comments
Closed
1 task done
Labels
enhancement New feature or request

Comments

@NotEvenANeko
Copy link

What's on your mind?

Mix this.configuration and _options instead of _config || this.configuration can make api easier for use.

For example I have a system which manage lots of apps, and I set userKey in the createConfiguration to manage apps. When I want to send notifications to one app, I can set the apiKey in the _options parameter, but I also need to provide parameters like baseServer, httpApi and middleware. Mix this.configuration and _options can make properties of the _options parameter optional.

Create a new DefaultApi instance every time is a way to solve this problem, but sometimes I want to make sure that only one instance exists, like using with NestJS.

Code of Conduct

  • I agree to follow this project's Code of Conduct
@NotEvenANeko
Copy link
Author

btw the link of the 'Code of Conduct' in the issue template is broken.

@emawby emawby added the enhancement New feature or request label Apr 28, 2023
@emawby
Copy link
Contributor

emawby commented Apr 28, 2023

Thank you for reporting, this request has been noted for future releases!

@nan-li nan-li closed this as completed Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants