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

* New config option to allow more customisation on the servers added … #726

Closed

Conversation

ProjectZero4
Copy link
Contributor

@ProjectZero4 ProjectZero4 commented Sep 13, 2023

…to the open api yaml

See the below screenshots to see how the generated yaml changes

OLD YAML

image

NEW YAML

image

@ProjectZero4
Copy link
Contributor Author

Just to explain the use case as I've just realised that the generated html and blade file that doesn't take advantage of this change.

We currently use a different provider for our api doc view but would like to use this package to start automating our generation of our yaml file

@shalvah
Copy link
Contributor

shalvah commented Sep 14, 2023

Sorry, gonna also close this, for the same reason as #729 . You can also achieve this with the openapi.overrides section. That's what it was added for! 😅

Also, small note: I always find it a bit funny how new contributors are quick to suggest a "new config item". There is such a thing as too many config items—it harms usability for users, increases maintenance overhead for maintainers (not to mention possibly multiplying the number of code paths). Every one wants to add a new config item for their use case, but if I accepted all these, and you as a new user installed this package and opened the config file to see 200 options, that would put you off.

So (even apart from the OpenAPI thing), my default reaction to anything proposing a new config item is No. There has to be a strong use case for it, a possibility that it will be valuable to more than just a few people, or the impossibility of achieving it any other way. Just letting you know. I'd say that, if you're thinking of adding a new config item, it's best to raise it as an issue for discussion first.

@shalvah shalvah closed this Sep 14, 2023
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