search

cakephp / localized

I18n and L10n related CakePHP code
Other
213 stars 179 forks source link

Lv validation #123

Closed makallio85 closed 8 years ago

dereuromark commented 8 years ago

Looks like we need to merge https://github.com/cakephp/localized/pull/122 first as those commits are on top of the changes there. But looks good.

makallio85 commented 8 years ago

I totally ended up messing these commits under wrong prs. Should I make totally new one that has these two different improvements combined?

ps. Is there any way to put one commit to other pull request? It seems that github automatically updated old pull request before I could do new one?

dereuromark commented 8 years ago

Its fine, once the other PR is merged, this will only show the actually new commits.