cds-snc / notification-planning

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

Change the SMS daily limits from fragments back to text messages #1298

Closed adriannelee closed 1 year ago

adriannelee commented 1 year ago

Description

As a Sender, I need to be able to track my daily limits without learning a new concept so that it's easy for me to stay under my limit.

WHY are we building? Because ‘daily limits’ is something we need to educate and create awareness around, educating Senders on ‘Fragments’ could make the design more challenging to execute well.

WHAT are we building?

VALUE created by our solution

Documentation and Artifacts

Good docs, figma mockups, ADRs, screenshots etc.

Acceptance Criteria

On authenticated pages, unauthenticated pages, API documentation, and API error messages, Senders won't see the word 'fragments'. Their daily SMS limit will be counted using 'text messages'

QA Steps

Ensure messages are updated in the following scenarios:

GC Articles Publish checklist

yaelberger-commits commented 1 year ago

Some data: Data question I’m hoping a dev can help me answer through a query: What is the average character length of SMS messages sent through Notify? How many SMS messages are 600 characters or more?

*doesn't factor in that some templates contain variables and might make the final SMS message longer

adriannelee commented 1 year ago

Started working on this today with Yedida and Phil, creating an inventory of the screens that would need to change when "fragments" is removed

adriannelee commented 1 year ago

Hey team! Please add your planning poker estimate with Zenhub @adriannelee @amazingphilippe @YedidaZalik

adriannelee commented 1 year ago

Yedida worked on the content for removing "Fragments" today and I did (a very quick) review. I think it's good for handoff? (other than translation)

yaelberger-commits commented 1 year ago

2 things next 1) Prototype is screenshots and we don't have the FR so will ask if we need to add more screenshots or get translations another way 2) Now using placeholder # for number of text messages (fragments to text messages) devs need to give us the number and number of characters 1,000 text messages is the new daily limit 612 is the new character limit (to confirm)

YedidaZalik commented 1 year ago

Discussed with Marie-Sophie and agreed I'd tag her in Figma where we need French. That's done now so it's ready for her translation.

smcmurtry commented 1 year ago

Translations done, could look at this for dev-design review today.

YedidaZalik commented 1 year ago

Discussed at dev-design review and implemented decisions from the review. Made the relevant changes to TOU EN and FR in GC Articles as Yael said that could go ahead before dev implementation. Worked with Phil to tidy up figma for dev implementation. We need to let Ramya and Jimmy P know that this is changing.

yaelberger-commits commented 1 year ago

@jzbahrai can you please update the checklist above so we can see that all AC is met?

amazingphilippe commented 1 year ago

Merged recent work about Daily limits and Email limits. in Figma

Even this isn't enough. Andrew mapped out all the states and conditions. We missed trial and live

Ok test cases so far:

Made a new pattern template to avoid this when working on the sending flows in the future: Private Zenhub Image

Can be found in Notify's Figma component library