Skip to content
This repository has been archived by the owner on Jun 24, 2020. It is now read-only.

Worksheet summary graph js #377

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

cfarm
Copy link
Contributor

@cfarm cfarm commented Mar 7, 2015

Added

  • Loops for outputting graph lines for goals, risks, and red flags

Todo

  • Don't output graph container if there are no results

Screenshot

screen shot 2015-03-07 at 5 11 20 pm

@cfarm
Copy link
Contributor Author

cfarm commented Aug 28, 2015

@amymok ready for review

@cfarm
Copy link
Contributor Author

cfarm commented Aug 28, 2015

note on this: you can't see the code locally anymore because we removed the prepare worksheets JS from the Gruntfile so it's not compiled (because of our grunt deploy task errors)

@cfarm
Copy link
Contributor Author

cfarm commented Aug 28, 2015

to build this locally you can uncomment these 2 lines in Gruntfile.js

https://github.com/cfpb/owning-a-home/blob/master/Gruntfile.js#L119

https://github.com/cfpb/owning-a-home/blob/master/Gruntfile.js#L152

should be tested with grunt deploy task too to make sure that doesn't break if you decide to merge.

cc @virginiacc

@amymok
Copy link
Contributor

amymok commented Aug 28, 2015

I am not seeing the summary graph coming out.
screen shot 2015-08-28 at 6 23 00 pm

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants