Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Untimely evidence #9598

Merged
merged 38 commits into from
Feb 22, 2019
Merged

Untimely evidence #9598

merged 38 commits into from
Feb 22, 2019

Conversation

youngfreezyVA
Copy link
Contributor

Resolves #8340

Description

in the submit view, add an option to allow attorneys to indicate that untimely evidence was submitted. per this mock:
image

Acceptance Criteria

  • Code compiles correctly

Testing Plan

Try checking the untimely evidence checkbox on submit, submit to a judge
go to that judge, return to an attorney
go to that attorney, when in the submit view again, untimely evidence should still be checked
for ama attorney checkout only

screenshots of changes:
image
image
image

arrow icon doesn't exist yet, thus the reason it isn't there atm.

Copy link

@anyakhvost anyakhvost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should limit this checkbox to ama only.

@youngfreezyVA
Copy link
Contributor Author

We should limit this checkbox to ama only.

done!

@youngfreezyVA youngfreezyVA added the Ready-to-Merge This PR is ready to be merged and will be picked up by va-bot to automatically merge to master label Feb 22, 2019
@va-bot va-bot merged commit 661fb3c into master Feb 22, 2019
@va-bot va-bot deleted the Untimely-Evidence branch February 22, 2019 19:10
@ghost ghost removed the In-Progress label Feb 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready-to-Merge This PR is ready to be merged and will be picked up by va-bot to automatically merge to master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants