B-19728-INT customer upload additional docs #12939
Conversation
…naldocuments isn't complete
|
paulstonebraker
added
G-Unit
paulstonebraker
changed the title
|
Shouldn't there be a checkin for the FF in the milmove-feature-flags repository also? or was that already done? |
|
Reviews on pause while I address bug |
paulstonebraker
force-pushed
the
B-19728-2-INT
branch
from
9116e08
to
c9ea504
Compare
| @@ -133,6 +133,9 @@ export FEATURE_FLAG_MOVE_LOCK=false | |||
| export FEATURE_FLAG_OKTA_DODID_INPUT=false | |||
| export FEATURE_FLAG_SAFETY_MOVE=true | |||
|
|
|||
| # Feature flag for additional supporting documents uploaded by customer | |||
| export FEATURE_FLAG_MANAGE_SUPPORTING_DOCS=true | |||
There was a problem hiding this comment.
I don't see this FF in the feature flag repo
There was a problem hiding this comment.
Thanks, Pam reminded me on Friday but I forgot amidst bug fix found during her review. Will push to ff repo shortly
pkg/handlers/internalapi/uploads.go
Outdated
| return handlers.ResponseForError(appCtx.Logger(), e), e | ||
| } | ||
|
|
||
| if e != nil { |
There was a problem hiding this comment.
this is duplicate from the if statement above, correct?
There was a problem hiding this comment.
Yep good eye! Removed
Agility ticket
FF Repo PR
Summary
Give the customers the ability to upload documents without affecting their workflow!
This work create a new button for customers that appears on their MoveHome page after a move is submitted, that allows them to provide additional documentation to the government without affecting their workflow. We added a new column to the moves table that houses additional_documents_id. Customer have the ability to upload documents and delete documents after they've been uploaded.
How to test