Skip to content

Add a mapping paths #1271

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
Jul 21, 2017

Conversation

meyerbaptiste
Copy link
Member

Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets N/A
License MIT
Doc PR N/A

Copy link
Member

@soyuka soyuka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow! Thanks for the deprecation test on configuration btw 👍

@@ -109,16 +108,10 @@ public function getConfigTreeBuilder()
->end()
->end()

->arrayNode('loader_paths')
->arrayNode('mapping')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a BC break.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wasn't this changed on master (ie not in 2.x)?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why? This feature is not in a stable release.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh yes! ❤️

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My bad...

Copy link
Member

@dunglas dunglas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Feel free to merge when you think it's ready!

@meyerbaptiste meyerbaptiste merged commit 473fc04 into api-platform:master Jul 21, 2017
@meyerbaptiste meyerbaptiste deleted the add_mapping_paths branch July 21, 2017 13:03
fabpot added a commit to symfony/recipes that referenced this pull request Jul 21, 2017
… mapping paths (meyerbaptiste)

This PR was merged into the master branch.

Discussion
----------

[API Platform] Update the API Platform config to use the new mapping paths

| Q             | A
| ------------- | ---
| License       | MIT

We recently changed the way to configure resources' paths in API Platform 2.1.0 beta 3, see api-platform/core#1271.

Commits
-------

8a76aa5 Update the API Platform config to use the new mapping paths
hoangnd25 pushed a commit to hoangnd25/core that referenced this pull request Feb 23, 2018
markovlatkovic pushed a commit to markovlatkovic/recipes that referenced this pull request May 24, 2024
… mapping paths (meyerbaptiste)

This PR was merged into the master branch.

Discussion
----------

[API Platform] Update the API Platform config to use the new mapping paths

| Q             | A
| ------------- | ---
| License       | MIT

We recently changed the way to configure resources' paths in API Platform 2.1.0 beta 3, see api-platform/core#1271.

Commits
-------

8a76aa5 Update the API Platform config to use the new mapping paths
mo-melvin77 added a commit to mo-melvin77/recipes that referenced this pull request Jun 9, 2025
… mapping paths (meyerbaptiste)

This PR was merged into the master branch.

Discussion
----------

[API Platform] Update the API Platform config to use the new mapping paths

| Q             | A
| ------------- | ---
| License       | MIT

We recently changed the way to configure resources' paths in API Platform 2.1.0 beta 3, see api-platform/core#1271.

Commits
-------

8a76aa5 Update the API Platform config to use the new mapping paths
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.

5 participants