Skip to content

[Slim] Official htaccess#838

Merged
wing328 merged 2 commits intoOpenAPITools:masterfrom
ybelenko:slim_htaccess
Aug 21, 2018
Merged

[Slim] Official htaccess#838
wing328 merged 2 commits intoOpenAPITools:masterfrom
ybelenko:slim_htaccess

Conversation

@ybelenko
Copy link
Copy Markdown
Contributor

PR checklist

  • Read the contribution guidelines.
  • Ran the shell script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run ./bin/{LANG}-petstore.sh and ./bin/security/{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\.
  • Filed the PR against the correct branch: master, 3.3.x, 4.0.x. Default: master.
  • Copied the technical committee to review the pull request if your PR is targeting a particular programming language.

Description of the PR

I've swapped .htaccess with example from Slim - Apache configuration doc.
Of course, I've checked few endpoints, looks like routing works correct.

@jebentier @dkarlovi @mandrean @jfastnacht @ackintosh

@wing328 wing328 merged commit d374e1c into OpenAPITools:master Aug 21, 2018
@ybelenko ybelenko deleted the slim_htaccess branch August 21, 2018 04:54
@wing328 wing328 added this to the 3.2.2 milestone Aug 22, 2018
A-Joshi pushed a commit to ihsmarkitoss/openapi-generator that referenced this pull request Feb 27, 2019
* [Slim] Change htaccess to official

Slim docs contains more simple htaccess example.
https://www.slimframework.com/docs/v3/start/web-servers.html#apache-configuration

* [Slim] Refresh samples
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants