Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DMPRoadmap/roadmap#562 #633

Merged
merged 1 commit into from
Sep 12, 2017
Merged

Conversation

jollopre
Copy link
Contributor

TODO refactoring of guidances and notes.

-edit phases using plan navigation layout

-bootstrap plan progress. bootstrap sections header. added togglable section icons for expand/collapse

-answers_controller returning a JSON with partial views rendered. Split question types in different partials. Started new answers/status.js

-Rename view to edit_plan_answers.html.erb. Load tinymce through ES6 for question answers. Delete old status.js ES5. Added timeago to vendor

-fixed broken test regarding answers_controller

edit phases using plan navigation layout

bootstrap plan progress. bootstrap sections header. added togglable section icons for expand/collapse

answers_controller returning a JSON with partial views rendered. Split question types in different partials. Started new answers/status.js

Rename view to edit_plan_answers.html.erb. Load tinymce through ES6 for question answers. Delete old status.js ES5. Added timeago to vendor

fixed broken test regarding answers_controller
@jollopre jollopre requested a review from briri September 11, 2017 16:12
@briri
Copy link
Contributor

briri commented Sep 12, 2017

looks good

@briri briri merged commit 25426cb into DMPRoadmap:CDL-MVP Sep 12, 2017
@jollopre jollopre deleted the issue562 branch September 18, 2017 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants