Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Endpoint for listing generated notifications with results #267

Closed
13 tasks done
Tracked by #148
SandGrainOne opened this issue Oct 10, 2023 · 2 comments
Closed
13 tasks done
Tracked by #148

Endpoint for listing generated notifications with results #267

SandGrainOne opened this issue Oct 10, 2023 · 2 comments
Assignees
Labels
kind/user-story Used for issues that describes functionality for our users.

Comments

@SandGrainOne
Copy link
Member

SandGrainOne commented Oct 10, 2023

Description

swagger: https://app.swaggerhub.com/apis/STEPHANIEBUADU/Notifications/1.0

Tasks

  • Set up suggestion in swagger
  • QA of endpoint design
  • Implementation
  • K6 tests
  • QA
  • Manual testing

Consideration

  • We improve performance if we don't include the senders reference in the response. Is it important to include this value or could we drop it?

  • What behavior do we want if the user is not authorized to review the notifications e.g. it is not the creator requesting the notifications. Do we want to return 403 or could we simply return:

{
    "orderId": "guid from request path"
    "notifications": [],
    "generated": 0,
    "succeeded": 0
}

An important note here, is that this would also be the respose if you call this endpoint before the requested send time of the order.

  • Success results are: succeeded (accepted by communication services) and delivered. Meaning the success count might change if anything bounced back for instance. Is this ok?

Test

Acceptance Criteria

  • 200 OK response if orderId exists and user has access
  • 404 Not found if orderId does not exist
  • 404 NOT found if orderId exists, but user does not have access
  • 403 response if missing access token or scopes
  • 401 if missing bearer token
  • Senders reference should be included in response if it exists
  • Expected mapping between status and descriptions
@acn-sbuad acn-sbuad added the kind/user-story Used for issues that describes functionality for our users. label Oct 10, 2023
@acn-sbuad acn-sbuad self-assigned this Nov 3, 2023
@acn-sbuad
Copy link
Contributor

Current state of swagger is a starting point for discussing the design of the endpoint. Considerations with current state are

  • how to see what is sent to a specific user across notification channels

@acn-sbuad
Copy link
Contributor

acn-sbuad commented Nov 7, 2023

/notifications/email endpoint is ready for implementation. Swagger is up to date and can be used as reference

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/user-story Used for issues that describes functionality for our users.
Projects
None yet
Development

No branches or pull requests

2 participants