Skip to content

Conversation

@markstory
Copy link
Member

Include an example of usage in View Cells. Does this look reasonable @ADmad?

Refs cakephp/cakephp#10718

Include an example of usage in View Cells.

Refs cakephp/cakephp#10718
@markstory markstory added this to the 3.x milestone Jun 19, 2017
'scope' => 'favourites'
]
);
$this->set('favorites', $results);
Copy link
Member

Choose a reason for hiding this comment

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

You used a different spelling of "favorites" on line 237 😄

@ADmad
Copy link
Member

ADmad commented Jun 19, 2017

@markstory Looks good except for the minor issue jeremyharris pointed out. Thanks for taking care of docs updation.

@ADmad
Copy link
Member

ADmad commented Jun 19, 2017

I updated to consistently use the US spelling 😛

@ADmad
Copy link
Member

ADmad commented Jun 19, 2017

Can this be merged or do we wait for FR and JA translations? (Dunno what's the process followed on docs PRs).

@HavokInspiration
Copy link
Member

No need to wait for translations.
The tags are just here to help us keep track of what's left to be translated.

Copy link
Member

@ravage84 ravage84 left a comment

Choose a reason for hiding this comment

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

Fixed some syntax errors, but looks good. Thanks!

@ADmad ADmad merged commit 158cf81 into 3.next Jun 19, 2017
@ADmad ADmad deleted the 3next-paginator branch June 19, 2017 16:42
HavokInspiration added a commit that referenced this pull request Jul 4, 2017
okinaka added a commit to okinaka/docs that referenced this pull request Aug 5, 2017
chinpei215 added a commit that referenced this pull request Aug 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants