cds-snc / notification-planning

Project planning for GC Notify Team
4 stars 0 forks source link

Provide more reassurance for sending bulk messages #192

Closed yaelberger-commits closed 2 years ago

yaelberger-commits commented 2 years ago

Description As a Notify user, I need to be able to confirm that my bulk message is being sent to all the recipients on my list so that I can have confidence I sent my messages to the right number of people.

WHY are we building? 5/9 people not sure when the email would be sent. They wanted to see the preview earlier (the review page). 3 / 9 participants were unsure from the preview that the bulk email would go to all the recipients (the preview only shows one email)

WHAT are we building?

VALUE created by our solution. A clearer screen and users that have confidence when sending bulk messages.

Acceptance Criteria (Definition of done)

Given a user has uploaded a list of recipients, when they are about to click the final "send" button, then the user has confidence their messages will be sent to all the recipients on their list.

amazingphilippe commented 2 years ago

Follow design specs in Figma https://www.figma.com/file/fRVRKm7cM8MUuBWUJbrfgU/?node-id=4165%3A19118

EliseKa commented 2 years ago

Phil, Yedida and I started documenting the problem and options yesterday in FigJam

EliseKa commented 2 years ago

@amazingphilippe, @YedidaZalik and I have designed options in Figma

amazingphilippe commented 2 years ago

Being reviewed:

amazingphilippe commented 2 years ago

Figma prototype ready for testing with users.

FR Link: https://www.figma.com/proto/fRVRKm7cM8MUuBWUJbrfgU/Sending-from-web-interface-%7C-Envoyer-par-interface-web?page-id=4474%3A19571&node-id=4513%3A20328&viewport=342%2C48%2C0.3&scaling=scale-down&starting-point-node-id=4513%3A20328&show-proto-sidebar=1

EN Link: https://www.figma.com/proto/fRVRKm7cM8MUuBWUJbrfgU/Sending-from-web-interface-%7C-Envoyer-par-interface-web?page-id=4474%3A19571&node-id=4513%3A20277&viewport=342%2C48%2C0.3&scaling=scale-down&starting-point-node-id=4513%3A20277&show-proto-sidebar=1

adriannelee commented 2 years ago

Noting that this card needs to be turned into an implementation card

amazingphilippe commented 2 years ago

A11y annotations: https://www.figma.com/file/fRVRKm7cM8MUuBWUJbrfgU/Sending-from-web-interface-%7C-Envoyer-par-interface-web?node-id=4914%3A21214

Ready for dev-design review

adriannelee commented 2 years ago

Dev-design review happening today.

yaelberger-commits commented 2 years ago

Hey team! Please add your planning poker estimate with ZenHub @amazingphilippe @sastels

amazingphilippe commented 2 years ago

PR Draft for implementation https://github.com/cds-snc/notification-admin/pull/1239

yaelberger-commits commented 2 years ago

Review criteria in PR

yaelberger-commits commented 2 years ago

Need someone to scan the page for a117 check and check against the figma

yaelberger-commits commented 2 years ago

Yael to QA against Figma Adrianne to QA for A11y scan

adriannelee commented 2 years ago

Reviewed for a11y QA and left a few comments

yaelberger-commits commented 2 years ago

QA: https://github.com/cds-snc/notification-admin/pull/1239

To access these templates:

On an email and a sms template Add multiple recipients and add one of the following csv A good csv A csv with bad emails or bad phone numbers A csv with the wrong headers Review the pages. List should be limited to the first 10 entries. If your file is valid, you'll be able to load more entries for preview If your file is invalid, you wont see a preview Moving from previous to next preview should be allowed, the first email only has a "next preview" button the last preview only has a "previous preview" button

yaelberger-commits commented 2 years ago

My QA is done @amazingphilippe @adriannelee do we need to add Figma links to Airtable for this one?

yaelberger-commits commented 2 years ago

Adrianne final a11y review

adriannelee commented 2 years ago

Did the last a11y review, and updated the records on Airtable. The records on Airtable weren't related to a hypothesis

yaelberger-commits commented 2 years ago

@amazingphilippe is this in prod now?

amazingphilippe commented 2 years ago

I need to fix some conflicts and then I can merge this to staging. It will become production the next time we do a release

yaelberger-commits commented 2 years ago

Has been merged, ready for next release