Skip to content

Conversation

dunglas
Copy link
Member

@dunglas dunglas commented Jul 4, 2016

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

* @author Paul Le Corre <paul@lecorre.me>
*/
class UnderscoreResourcePathGenerator implements ResourcePathGeneratorInterface
final class DashPathNamingStrategy implements PathNamingStrategyInterface
Copy link
Contributor

Choose a reason for hiding this comment

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

\o/

@teohhanhui
Copy link
Contributor

teohhanhui commented Jul 5, 2016

I think it should be ApiPlatform\Core\Naming\ResourcePathNamingStrategyInterface::generateResourceBasePath.

@dunglas dunglas force-pushed the resource_naming_strategy branch from 081ff95 to 4b5a974 Compare July 7, 2016 06:22
@dunglas dunglas force-pushed the resource_naming_strategy branch from 4b5a974 to 3573613 Compare July 7, 2016 06:26
@dunglas dunglas merged commit 7825afb into master Jul 7, 2016
@dunglas dunglas deleted the resource_naming_strategy branch July 7, 2016 07:30
magarzon pushed a commit to magarzon/core that referenced this pull request Feb 12, 2017
…_strategy

Rename Path Generator to Path Naming Strategy
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.

3 participants