Update charts to use new study level summary fields

Issue #768 resolved
Ed McDonagh created an issue

Some are done already in issue #759 / Pull Request #315. Some, such as those using total DAP to be done afterwards.

Comments (10)

  1. Ed McDonagh reporter

    Merged in issue759AsManyTasks (pull request #315)

    Fixes #759 Summary fields in GeneralStudyModuleAttr, Refs #768 Update charts to use new study level summary fields

    Approved-by: David Platten

    Thanks @dplatten and @LuukO for reviewing (and adding to!) I'm leaving the branch open as you are working in it David - not sure if you want to start a new branch to finish ref #768 off, from develop?

    → <<cset 890828d285b3>>

  2. Ed McDonagh reporter

    Merged in issue759AsManyTasks (pull request #315)

    Fixes #759 Summary fields in GeneralStudyModuleAttr, Refs #768 Update charts to use new study level summary fields

    Approved-by: David Platten

    Thanks @dplatten and @LuukO for reviewing (and adding to!) I'm leaving the branch open as you are working in it David - not sure if you want to start a new branch to finish ref #768 off, from develop?

    → <<cset 890828d285b3>>

  3. David Platten

    I think that all the charts that can make use of the study level summary fields are now doing so. Are you happy for me to update the changes files in “develop” to reflect this?

  4. Ed McDonagh reporter

    If the only files to change are the changes files, then yes. Close the issue with the commit.

  5. Log in to comment