Skip to content

Conversation

Simperfit
Copy link
Contributor

Q A
Bug fix? no
New feature? no
BC breaks? yes
Deprecations? no
Tests pass? yes
Fixed tickets #537
License MIT

This is resolving the problem of resource being a soft reserved word in php7

@dunglas
Copy link
Member

dunglas commented May 15, 2016

Thanks! Can you update the doc as accordingly?

@dunglas dunglas merged commit 515f56a into api-platform:master May 15, 2016
@dunglas
Copy link
Member

dunglas commented May 15, 2016

I propose to also rename @Property to @ApiProperty for coherence. Do you agree @api-platform/core-team @Simperfit?

@theofidry
Copy link
Contributor

Makes sense 👍

@Simperfit
Copy link
Contributor Author

I will do it and update the doc

@Simperfit Simperfit deleted the fix-issue-537 branch May 16, 2016 15:28
magarzon pushed a commit to magarzon/core that referenced this pull request Feb 12, 2017
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