https://redmine.gc.cuny.edu/https://redmine.gc.cuny.edu/favicon.ico2017-08-31T20:29:49ZCUNY Graduate Center - Project Tracking SystemCUNY Academic Commons - Feature #8637: Remove extra fields from signup processhttps://redmine.gc.cuny.edu/issues/8637?journal_id=335272017-08-31T20:29:49ZDaniel Jonesdrjones18@gmail.com
<ul></ul><p>This was pretty straightforward (I did the 2nd option). I do have one question though: should I delete the code that currently outputs the "Blog Details" section of the registration page, which allows users to create a new blog when they register? There was also some code related to the "College" field that had it also show the Positions field that I've just commented out for now, but will delete if you say it's alright.</p>
<p>Once I know what to do on those two questions, I'll go ahead and make the final changes and push the commit.</p> CUNY Academic Commons - Feature #8637: Remove extra fields from signup processhttps://redmine.gc.cuny.edu/issues/8637?journal_id=335322017-09-01T02:03:07ZBoone Gorgesboone@gorg.es
<ul></ul><p>If it's not shown in the mockup, feel free to rip it out of the template. (As you can tell, that template has been accruing junk for years.) Thanks!</p> CUNY Academic Commons - Feature #8637: Remove extra fields from signup processhttps://redmine.gc.cuny.edu/issues/8637?journal_id=335412017-09-01T13:41:45ZDaniel Jonesdrjones18@gmail.com
<ul></ul><p>Great! Pushed some changes here: <a class="external" href="https://github.com/cuny-academic-commons/cac/commit/b3c97f7c70a1b2ee455e94336e5f37c925853b73">https://github.com/cuny-academic-commons/cac/commit/b3c97f7c70a1b2ee455e94336e5f37c925853b73</a></p>
<p>Right now it still splits the fields into "Account Details" and "Profile Details", with the Full Name field on the right under "Profile Details", which is different from the mock-up. Should I work on that or is that for a different ticket, and can leave this for now?</p> CUNY Academic Commons - Feature #8637: Remove extra fields from signup processhttps://redmine.gc.cuny.edu/issues/8637?journal_id=335532017-09-01T19:01:30ZBoone Gorgesboone@gorg.es
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Resolved</i></li></ul><p>Ah, I hadn't noticed that change. I assume Chris made it that way because it sorta makes more sense to think of it as "account" info rather than "profile" info. However, I understand that it's techincally profile data, and that it's not a straightforward task to switch columns.</p>
<p>I'm going to make the call to leave it as-is - we'll deviate from the mock-up in this respect. I've copied Chris here in case he feels strongly about the move.</p>