SeedCompany / cord-field

CORD UI

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

QR - Users can submit reports inadvertently with no data

sync-by-unito opened this issue · comments

┆Issue is synchronized with this Monday item by Unito

sheri_jennings@tsco.org commented:

Users can submit reports without entering any information. A notification should alert the user if they are submitting a report that has missing information.

"Are you sure you want to submit this report? The following was not updated or left blank"
Team News
Story
Progress

The error message should be triggered by role:
Partner would only receive message if partner variant or Story is left blank
FPM would only receive message if Field variant, Story was left blank, or PnP not uploaded
Comms would receive message if comms variant was left blank

sheri_jennings@tsco.org commented:

(Amy) QR - Since there is no review your work step, you can submit your report inadvertantly with no data in it, especially if they don't save work. Or they can have wrong data and not know it. Or if there is a bug and the data didn't save.

➤ Seth McKnight commented:

@sheri_jennings@tsco.org  ( https://seed-company-squad.monday.com/users/36490858-sheri_jennings-tsco-org )(cc @amy Rickwartz ( https://seed-company-squad.monday.com/users/34244765-amy-rickwartz ) )... this was from feedback from the initial Quarterly Reports testing. Would you do a quick review and validate if

  1. we won't do this and it should be archived
  2. this is a real need but shouldn't be addressed this cycle
  3. or this is a real need and should be here

sheri_jennings@tsco.org commented:

@Vonshel Davis ( https://seed-company-squad.monday.com/users/45522041-vonshel-davis )  I am looking for your response on this ticket, did you open a separate ticket? I know I saw you post your rec based on your research

➤ Amy Rickwartz commented:

In our product/dev sync, we decided to work on revising this design because the pop-up modal covers the confetti interaction on submit form. Created a sub-task for work.

➤ Seth McKnight commented:

@andre Turner  ( https://seed-company-squad.monday.com/users/37419707-andre-turner )I know this is already In Review, but would you add story points when you get a chance?

Note on Status - In Review

➤ Vonshel Davis commented:

@sheri_jennings@tsco.org ( https://seed-company-squad.monday.com/users/36490858-sheri_jennings-tsco-org ) I've completed the 1st draft for ticket and ready for review. I'm wondering if you have anytime available between 2pm - 5pm EST today. I've attached a quick mockup for review to this message.

https://seed-company-squad.monday.com/protected_static/12597801/resources/974098296/big-Quarterly%20Report%20-%20Submit%20Report%20Modal%20E2.jpg

sheri_jennings@tsco.org commented:

@carson Full ( https://seed-company-squad.monday.com/users/36973420-carson-full ) @andre Turner ( https://seed-company-squad.monday.com/users/37419707-andre-turner )  Please deploy to dev. We have decided to go with the original design/code that has been completed for now and will revisit inline design separately. Understanding the confetti after ignoring missing data message is not ideal, we would like this deployed for quarterly reporting that starts next week