Skip to content

added requirements for {_format} #11303

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

Merged
merged 1 commit into from
Apr 6, 2019
Merged

added requirements for {_format} #11303

merged 1 commit into from
Apr 6, 2019

Conversation

stoccc
Copy link
Contributor

@stoccc stoccc commented Apr 5, 2019

I added requirements for the {_format} parameter: in this example _format should be xml or html

I added requirements for {_format}: in this example `_format` should be `xml` or `html`
@wouterj
Copy link
Member

wouterj commented Apr 6, 2019

Great! This looks like a good addition to me. Thanks for submitting the PR, @stoccc!

@wouterj wouterj merged commit f2ebfed into symfony:3.4 Apr 6, 2019
wouterj added a commit that referenced this pull request Apr 6, 2019
This PR was merged into the 3.4 branch.

Discussion
----------

added requirements for {_format}

I added requirements for the `{_format}` parameter: in this example `_format` should be `xml` or `html`

<!--

If your pull request fixes a BUG, use the oldest maintained branch that contains
the bug (see https://symfony.com/roadmap for the list of maintained branches).

If your pull request documents a NEW FEATURE, use the same Symfony branch where
the feature was introduced (and `master` for features of unreleased versions).

-->

Commits
-------

f2ebfed added requirements for {_format}
@wouterj wouterj added the ⭐️ EU-FOSSA Hackathon https://symfony.com/blog/the-symfony-and-api-platform-hackathon-is-coming label Apr 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⭐️ EU-FOSSA Hackathon https://symfony.com/blog/the-symfony-and-api-platform-hackathon-is-coming Status: Reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants