codeforamerica / clean

Apply for CalFresh in SF
http://demo.cleanassist.org
MIT License
20 stars 7 forks source link

Initial release: document uploader #504

Closed daguar closed 9 years ago

daguar commented 9 years ago

This is a checklist issue encompassing the things we want to do prior to release (we will have cleanup after, but we should ship ASAP):

Release to do

Stretch goal = release by EOW!

lippytak commented 9 years ago

Adding note about RM training

daguar commented 9 years ago

Progress bar styling done! Thanks @alanjosephwilliams

daguar commented 9 years ago

Fixed the URL in email

daguar commented 9 years ago

I added to the release list "Printable confirmation page (and/or copy that makes sense with doc upload done) ( #355 )" based on conversation in that issue.

daguar commented 9 years ago

Friendly error messages done, w0000t! :fist: @alanjosephwilliams

alanjosephwilliams commented 9 years ago

styles for #494 pushed, moving on to confirmation page

daguar commented 9 years ago

:+1: merged!

lippytak commented 9 years ago

Pinged RM with sample email. Will check in tomorrow.

BTW @daguar can we make their username sfhsa?

daguar commented 9 years ago

@lippytak :+1: to making username sfhsa (though now it's public :P )

lippytak commented 9 years ago

Just successfully tested 10, 20, and 40 'take a photo' uploads on mobile. Looks good :+1: ! screen shot 2015-04-27 at 8 30 07 am

One upload got stuck on 20 docs but eventually errored out with the 'bad file type' error message (wrong). I'll re-open the other issue and give @daguar a timestamp to look into it.

lippytak commented 9 years ago

Removing the BG release requirement from OG comment above: ' PDFs with lots of pages = request timeout at end (but backend is okay) — may require background job implementation ( #496 )'

alanjosephwilliams commented 9 years ago

Woo wooooo

On Mon, Apr 27, 2015 at 8:35 AM, Jake Solomon notifications@github.com wrote:

Removing the BG release requirement from OG comment above: ' PDFs with lots of pages = request timeout at end (but backend is okay) — may require background job implementation ( #496 https://github.com/codeforamerica/clean/issues/496 )'

— Reply to this email directly or view it on GitHub https://github.com/codeforamerica/clean/issues/504#issuecomment-96712635 .

t: @alanjosephwilli p: 817 713 6264

daguar commented 9 years ago

Removed "printable confirmation page" issue reference and replaced with "Change final page (new page/route) to have text that reflects that the user has already uploaded documents" per our collective understanding

daguar commented 9 years ago

Updated to add

daguar commented 9 years ago

Checked off final page confirmation. Thanks @alanjosephwilliams!

daguar commented 9 years ago

Added deployment tasks

daguar commented 9 years ago

Started on deployment tasks

daguar commented 9 years ago

Credentials are stored in the private creds doc

daguar commented 9 years ago

Successfully tested on demo

daguar commented 9 years ago

Live on dev, demo, and production. :shipit: :+1:

@lippytak — Last task is release notes to users, and then this is good to close!

lippytak commented 9 years ago

Release email sent to all users.

@daguar do a final gut check but I think we're :+1: to close this.

daguar commented 9 years ago

:shipit:

daguar commented 9 years ago

Some stats so far: