Skip to content
This repository has been archived by the owner on Jan 23, 2024. It is now read-only.

Implement e-signature #375

Closed
10 tasks done
lippytak opened this issue Feb 24, 2015 · 5 comments · Fixed by #416
Closed
10 tasks done

Implement e-signature #375

lippytak opened this issue Feb 24, 2015 · 5 comments · Fixed by #416
Assignees

Comments

@lippytak
Copy link
Contributor

  • Remove current jSignature code
  • Replace signature view with form inputs for name and 'agree' checkbox
  • Add migration
  • Save signature data to the DB (name, checkbox status)
  • Print signature to CF-285
  • Print signature to ROI
  • Write tests
  • Update cover page to explicitly address that the sig may look weird but it's valid
  • Review with Leo
  • Rebase and PR

Research: #326

Working on: https://github.com/codeforamerica/clean/tree/e-signature

@lippytak lippytak self-assigned this Feb 24, 2015
@lippytak lippytak added the ready label Feb 24, 2015
@lippytak
Copy link
Contributor Author

^ updated with the steps to implement

@daguar
Copy link
Contributor

daguar commented Feb 28, 2015

Maybe we review with Leo first? Based on current level of EW confusion, I think it makes sense to check in first before putting in the effort — it may change the requirements.

How did the Intuit applications deal with this, btw?

@daguar
Copy link
Contributor

daguar commented Mar 2, 2015

Moved this to current sprint — Jake, are you doing the front end stuff here or am I? I know some of the backend stuff I need to do, so let's just touch base and break out into separate issues if necessary.

@lippytak
Copy link
Contributor Author

lippytak commented Mar 3, 2015

Will be working on here: https://github.com/codeforamerica/clean/tree/e-signature. All I've done so far is delete all the jSignature remnants.

@daguar
Copy link
Contributor

daguar commented Mar 3, 2015

I rebased against master, so the e-sig working branch now has all the backend storage work in it.

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 a pull request may close this issue.

2 participants