Skip to content

Issue634 - #635

Merged
briri merged 3 commits into
DMPRoadmap:developmentfrom
DigitalCurationCentre:issue634
Sep 12, 2017
Merged

Issue634#635
briri merged 3 commits into
DMPRoadmap:developmentfrom
DigitalCurationCentre:issue634

Conversation

@jollopre

@jollopre jollopre commented Sep 12, 2017

Copy link
Copy Markdown
Contributor

Optimisation made at two levels:

  • Retrieving only sections, questions... related to the phase selected
  • Iterating over the questions related to that phase instead of all the plan questions to create the hash of question_guidance.
@jollopre
jollopre requested a review from xsrust September 12, 2017 13:04
@briri

briri commented Sep 12, 2017

Copy link
Copy Markdown
Contributor

looks much more efficient
@briri
briri merged commit cbe3cf0 into DMPRoadmap:development Sep 12, 2017
@jollopre
jollopre deleted the issue634 branch September 13, 2017 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants

@jollopre @briri @vyruss