bcgov / foi-flow

Freedom of Information modernization
Apache License 2.0
5 stars 3 forks source link

Notification for request in Call For Records #2306

Closed lmullane closed 2 years ago

lmullane commented 2 years ago

Assumptions & Scope What are the assumptions for this story?

When a request goes to a ministry in Call For Records, it is assigned to the ministry team, not an individual assignee. Currently, notifications are only sent to an individual who is the assignee or a watcher. This would be a change in that pattern. For discussion.

Should all ministry team members receive the notification?

What is IN scope?

Notifications to the ministry coordinator(s) when a request goes into Call For Records

What is NOT in scope?

Other notifications

System generated comments of the state change are already in place.

Acceptance Criteria

Scenario 1: Automated notification for unassigned Call for Records request

Scenario 2: Notifications auto-dismiss upon assignment

...

Dependencies? What is the impact of this dependency? (If so, link dependency in the ticket, make it visible in a team´s backlog)

Validation Rules? (If yes, list here)

Design @xxx - please link the Design here

Definition of Ready

  1. [ ] Is there a well articulated User Story?
  2. [ ] Is there Acceptance Criteria that covers all scenarios (happy/sad paths)?
  3. [ ] If there is a user interface, is there a design?
  4. [ ] Does the user story need user research/validation?
  5. [ ] Does this User Story needs stakeholder approval?
  6. [ ] Design / Solution accepted by Product Owner
  7. [ ] Is this user story small enough to be completed in a Sprint? Should it be split?
  8. [ ] Are the dependencies known/ understood? (technical, business, regulatory/policy)
  9. [ ] Has the story been estimated?

Definition of Done

  1. [ ] Passes developer unit tests
  2. [ ] Passes peer code review
  3. [ ] If there's a user interface, passes UX assurance
  4. [ ] Passes QA of Acceptance Criteria with verification in Dev and Test
  5. [ ] Confirm Test cases built and succeeding
  6. [ ] No regression test failures
  7. [ ] Test coverage acceptable by Product Owner
  8. [ ] Ticket ready to be merged to master or story branch
  9. [ ] Developer to list Config changes/ Update documents and designs
  10. [ ] Can be demoed in Sprint Review
  11. [ ] Tagged as part of a Release
  12. [ ] Feature flagged if required
  13. [ ] Change Management activities done?
lmullane commented 2 years ago

Should we automatically make all ministry coordinators a watcher?

m-prodan commented 2 years ago

Update to include scenario - notification auto dismisses for all when assigned.

m-prodan commented 2 years ago

AC updated per discussion at refinement.

Decision from refnement: not to auto-assign as watchers.

m-prodan commented 2 years ago

Updated AC per discussion with Aparna - since request ID is already showing on the notification itself, no need to put it again in description (AC 1 - removed request ID from the notification message)

m-prodan commented 2 years ago

Working as expected - QA Passed great work @aparna-aot