[GitHub] struts pull request #138: WW-3171 WW-3650 WW-4581: Locale aware converters

2017-05-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/struts/pull/138 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] struts issue #138: WW-3171 WW-3650 WW-4581: Locale aware converters

2017-05-18 Thread cnenning
Github user cnenning commented on the issue: https://github.com/apache/struts/pull/138 👍 for merging --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so,

[GitHub] struts issue #137: WW-4762: Default provider

2017-05-18 Thread lukaszlenart
Github user lukaszlenart commented on the issue: https://github.com/apache/struts/pull/137 Please see docs, I have renamed the constant and the class https://cwiki.apache.org/confluence/display/WW/Localization#Localization-Usingonlyglobalbundles --- If your project is set up for

[GitHub] struts pull request #137: WW-4762: Default provider

2017-05-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/struts/pull/137 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] struts issue #138: WW-3171 WW-3650 WW-4581: Locale aware converters

2017-05-18 Thread lukaszlenart
Github user lukaszlenart commented on the issue: https://github.com/apache/struts/pull/138 Any other objections? I would start with what we have here and improve --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] struts issue #137: WW-4762: Default provider

2017-05-18 Thread lukaszlenart
Github user lukaszlenart commented on the issue: https://github.com/apache/struts/pull/137 Any objections to merge this PR? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature