yeoman-app icon indicating copy to clipboard operation
yeoman-app copied to clipboard

Fix issue with radio button when choice is string and not object.

Open revathskumar opened this issue 8 years ago • 2 comments

related #74

screen shot 2016-03-28 at 11 02 39 am

screen shot 2016-03-28 at 11 11 33 am

revathskumar avatar Mar 28 '16 05:03 revathskumar

Thanks for picking this up @revathskuma! I had a look on your solution and noticed that your fix would work for RadioButton only (which is definitely an improvement) but, I would prefer a generic solution see #75.

My idea would be to normalize the data upfront, before we pass them to the render process https://github.com/yeoman/yeoman-app/blob/d58650cfc94a4523104f67392e64c59e7afcf02e/src/browser/yo/adapter.js#L6. It shouldn't be that hard, but changes in this area require a solid test coverage. Unfortunately we have a lack of tests in some areas. I'm more than happy to support and guide you through the required changes.

stefanbuck avatar Mar 28 '16 16:03 stefanbuck

Yes, I have seen #75. but since I am new to this code base I haven't tried for generic solution. But lemme try to work on it.

On Mon, Mar 28, 2016 at 9:41 PM, Stefan Buck [email protected] wrote:

Thanks for picking this up @revathskuma! I had a look on your solution and noticed that your fix would work for RadioButton only (which is definitely an improvement) but, I would prefer a generic solution see #75 https://github.com/yeoman/yeoman-app/issues/75.

My idea would be to normalize the data upfront, before we pass them to the render process https://github.com/yeoman/yeoman-app/blob/d58650cfc94a4523104f67392e64c59e7afcf02e/src/browser/yo/adapter.js#L6. It shouldn't be that hard, but changes in this area require a solid test coverage. Unfortunately we have a lack of tests in some areas. I'm more than happy to support and guide you through the required changes.

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/yeoman/yeoman-app/pull/109#issuecomment-202465103

with regards, Revath S Kumar, Rubyist / JavaScripter / PHP

markdownblogger.herokuapp.com

www.phprepo.in revathskumar.com +919995436867

http://twitter.com/revathskumar http://www.linkedin.com/in/revathskumar http://www.gplus.to/rsk http://www.about.me/revathskumar http://www.phprepo.in/ http://www.github.com/revathskumar http://www.youtube.com/revathskumar http://blog.revathskumar.com https://foursquare.com/revathskumar [image: normal-1.png] http://careers.stackoverflow.com/revathskumar

revathskumar avatar Mar 29 '16 06:03 revathskumar