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

Add Date of assessment to New functional score #68

Open
mgarzon opened this issue Aug 1, 2017 · 3 comments
Open

Add Date of assessment to New functional score #68

mgarzon opened this issue Aug 1, 2017 · 3 comments
Assignees
Milestone

Comments

@mgarzon
Copy link
Contributor

mgarzon commented Aug 1, 2017

Date of assessment should by default be the current date,.

@mgarzon mgarzon added this to the CWS - DEMO milestone Aug 1, 2017
@mgarzon mgarzon added the High label Aug 1, 2017
@kanwarbirsingh
Copy link
Contributor

kanwarbirsingh commented Aug 15, 2017

  1. No need for a default date.
  2. Add a red asterisk alongside the label.
  3. If the user clicks Sign button and date field is left empty, a popup box should open telling the user that the Date of Assessment field cannot be left empty.
  4. Format of the date value should be dd MMM yyyy instead of mm/dd/yyyy in Date of Assessment field.
  5. If the user clicks save button then no warning and the assessment date should be NULL.

@lpeyton
Copy link

lpeyton commented Nov 2, 2017

The problem is that currently it will not close if there is no date entered.

@lpeyton lpeyton assigned Eric-Alvarez and unassigned RANJODHHANJRA Nov 2, 2017
@Eric-Alvarez
Copy link
Contributor

Asterisk has been added to signify mandatory date.
Pop up box exists to tell user the Date of Assessment field cannot be left empty when signing or closing.
If a non-mandatory date when closing is to be pursued, the saving of the page must be changed to not require a date. Currently, when attempting to save the page an error is thrown if there is no date entered, causing the close and redirect to fail.

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

No branches or pull requests

6 participants