Skip to content

/ NelmioApiDocBundle

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

Jump to bottom

Add support for host in sandbox #277

Merged
merged 1 commit into from Nov 14, 2013
Merged

Add support for host in sandbox #277

merged 1 commit into from Nov 14, 2013

Conversation

@adriensamson
Copy link

@adriensamson adriensamson commented Nov 13, 2013

No description provided.

Adrien SAMSON
Add support for host in sandbox
f71ba12
@willdurand
Copy link
Collaborator

@willdurand willdurand commented Nov 13, 2013

what?
@adriensamson
Copy link
Author

@adriensamson adriensamson commented Nov 13, 2013

Hi,
In our project, we have many routes configured with different subdomains.
We'd like to have a central doc (docs.mydomain.tld/api/doc) with the sandbox doing crossrequests to all hosts configured in the project (articles.mydomain.tld/articles, authors.mydomain.tld/authors, ...).

This PR only makes the form action to contain the correct host.
@JJK801
Copy link

@JJK801 JJK801 commented Nov 13, 2013

👍 Helpful!
@KuiKui
Copy link

@KuiKui KuiKui commented Nov 13, 2013

👍
1 similar comment
@oziks
Copy link

@oziks oziks commented Nov 14, 2013

👍
willdurand added a commit that referenced this pull request Nov 14, 2013
@willdurand
Merge pull request #277 from adriensamson/sandbox-host
f94d640 
Add support for host in sandbox
@willdurand willdurand merged commit f94d640 into nelmio:master Nov 14, 2013
@willdurand
Copy link
Collaborator

@willdurand willdurand commented Nov 14, 2013

thanks.
@adriensamson adriensamson deleted the adriensamson:sandbox-host branch Nov 14, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
@adriensamson @willdurand @JJK801 @KuiKui @oziks