episphere / questionnaire

1 stars 2 forks source link

Message when required question is not answered #85

Closed joshid-ims closed 2 years ago

joshid-ims commented 2 years ago

The default message that shows up when any required question is not answered by user is:

image

But the document for Menstrual Cycle Survey says:

[error message if someone clicks NEXT without entering a date]: Please use the calendar to select the date that your next menstrual period started.

This is not something I can do or I do not know how to do it if it is possible to do. Is the default text or or can you please coordinate with Daniel and Brian on whether this feature is available?

danielruss commented 2 years ago

What module is this in?

joshid-ims commented 2 years ago

This is the new Menstrual Cycle Survey.

boyd-mj commented 2 years ago

Hi @danielruss any updates on this?

danielruss commented 2 years ago

looking

danielruss commented 2 years ago

So you want to change the message? This might create ugly(ier) markup.

[Q1!] Question with default message

Would something like this work? [Q2] |date|min=x max=y required| where a red box appears under the calendar and says "please fill out this field?"

joshid-ims commented 2 years ago

@huip-ims is that acceptable?

joshid-ims commented 2 years ago

Actually, this question is for @cusackjm. Is Daniel's suggestion acceptable? If so, do we need to update the Word document?

cusackjm commented 2 years ago

@joshid-ims just to clarify, this is the suggestion:

[SrvBlU_MENS2_v1r0] Please provide the first day that your next menstrual period started. This information helps researchers figure out where your body was in the menstrual cycle on the day you donated samples. If you do not remember the exact day, please make your best guess.

and on the next screen is: "<Calendar to indicate date /__/>" and this is required?

joshid-ims commented 2 years ago

@danielruss is that what you meant?

danielruss commented 2 years ago

image

danielruss commented 2 years ago

If this is ok, Just want to let you know you can use. Fix not pushed yet. Waiting to hear if this is ok.

|date| required min="5/9/2022" max="5/10/2022"|
cusackjm commented 2 years ago

@danielruss yes that option looks good

danielruss commented 2 years ago

Pushed. Please Test.

boyd-mj commented 2 years ago

@danielruss thanks! Just a heads up that since this is for the menstrual cycle survey we likely won't test for a bit given other priorities. We will make sure to update/close the issue once we do.

joshid-ims commented 2 years ago

The date widget is allowing invalid dates.

image

naiyume commented 2 years ago

@joshid-ims I have opened a new issue for this https://github.com/episphere/questionnaire/issues/107