Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Update parent_services for tip consistency#7585

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed
matthieu88160 wants to merge3 commits intosymfony:3.2frommatthieu88160:patch-3

Conversation

@matthieu88160
Copy link
Contributor

Currently the first page tip explain that the created service class extends the BaseDoctrineRepository but it is not represented by the code samples. Simply adding class declaration example for tip consistency and user reading improvement.

Currently the first page tip explain that the created service class extends the BaseDoctrineRepository but it is not represented by the code samples. Simply adding class declaration example for tip consistency and user reading improvement.
javiereguiluzand others added2 commitsMarch 7, 2017 11:22
Thank for typo ;)Two classes are presented.
@xabbuh
Copy link
Member

Thank you@matthieu88160.

xabbuh added a commit that referenced this pull requestJul 21, 2017
…, javiereguiluz)This PR was submitted for the 3.2 branch but it was merged into the 2.7 branch instead (closes#7585).Discussion----------Update parent_services for tip consistencyCurrently the first page tip explain that the created service class extends the BaseDoctrineRepository but it is not represented by the code samples. Simply adding class declaration example for tip consistency and user reading improvement.Commits-------45a44f3 Fixed a typo977142c Fixed a typo6855572 Update parent_services for tip consistency
xabbuh added a commit that referenced this pull requestJul 21, 2017
xabbuh added a commit that referenced this pull requestJul 21, 2017
@xabbuhxabbuh closed thisJul 21, 2017
xabbuh added a commit that referenced this pull requestJul 21, 2017
* 2.7:  Reworded the article about form login redirects  Explained the edge-case where the use_referer option doesn't work  [#7572] fix wording  [#7585] remove trailing whitespaces  [#7585] minor rewording  Fixed a typo  Fixed a typo  Update parent_services for tip consistency  [#7685] use the method role  Minor change  Updating doc to specify priority of default normalizer  [#7767] remove trailing space  Minor reword  Moved array validation to the Raw values sub-guide  Fixed some syntax issues  missing constraint example from the old readme  Update console dialog helper documentation  Improved the advantages/drawbacks of "controllers as services"
xabbuh added a commit that referenced this pull requestJul 21, 2017
* 2.8: (37 commits)  [#8192] use path() in PHP templates  Reworded the article about form login redirects  Explained the edge-case where the use_referer option doesn't work  [#7572] fix wording  [#7585] remove trailing whitespaces  [#7585] minor rewording  Fixed a typo  Fixed a typo  Update parent_services for tip consistency  [#7685] use the method role  Minor change  Updating doc to specify priority of default normalizer  [#7767] remove trailing space  [#7767] replace "options" with "entry_options"  [#7767] minor rewording  [#8047] add inline code comment  Fixed the issue in a different way  Jquery datePicker syntax update  [#8104] minor rewording  Add more precision about automatic provider assignation  ...
xabbuh added a commit that referenced this pull requestJul 21, 2017
* 3.2: (38 commits)  [#8192] use path() in PHP templates  Reworded the article about form login redirects  Explained the edge-case where the use_referer option doesn't work  [#7572] fix wording  [#7585] remove trailing whitespaces  [#7585] minor rewording  Fixed a typo  Fixed a typo  Update parent_services for tip consistency  [#7685] use the method role  Minor change  Updating doc to specify priority of default normalizer  [#7767] remove trailing space  [#7767] replace "options" with "entry_options"  [#7767] minor rewording  [#8047] add inline code comment  Fixed the issue in a different way  Jquery datePicker syntax update  Fix framework instantiation in event-dispatcher  [#8104] minor rewording  ...
xabbuh added a commit that referenced this pull requestJul 21, 2017
* 3.3: (46 commits)  [#8192] use path() in PHP templates  Reworded the article about form login redirects  Update Flex documentation with latest structure  Explained the edge-case where the use_referer option doesn't work  [#7572] fix wording  [#7585] remove trailing whitespaces  [#7585] minor rewording  Fixed a typo  Fixed a typo  Update parent_services for tip consistency  [#7685] use the method role  Minor change  Updating doc to specify priority of default normalizer  [#7767] remove trailing space  [#7767] replace "options" with "entry_options"  [#7767] minor rewording  [#8047] add inline code comment  Fixed the issue in a different way  Jquery datePicker syntax update  Fix framework instantiation in event-dispatcher  ...
xabbuh added a commit that referenced this pull requestJul 21, 2017
* 3.4: (48 commits)  [#8192] use path() in PHP templates  Reworded the article about form login redirects  Update Flex documentation with latest structure  Explained the edge-case where the use_referer option doesn't work  [#7572] fix wording  [#7585] remove trailing whitespaces  [#7585] minor rewording  Fixed a typo  Fixed a typo  Update parent_services for tip consistency  [#7685] use the method role  Minor change  Updating doc to specify priority of default normalizer  [#7767] remove trailing space  [#7767] replace "options" with "entry_options"  [#7767] minor rewording  [#8047] add inline code comment  Fixed the issue in a different way  Jquery datePicker syntax update  Fix framework instantiation in event-dispatcher  ...
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@javiereguiluzjaviereguiluzjaviereguiluz approved these changes

+1 more reviewer

@roman-1983roman-1983roman-1983 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Projects

None yet

Milestone

2.7

Development

Successfully merging this pull request may close these issues.

5 participants

@matthieu88160@xabbuh@javiereguiluz@roman-1983@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp