-
Notifications
You must be signed in to change notification settings - Fork 578
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
Feature: Notifications flow diagrams recreation #7470
base: master
Are you sure you want to change the base?
Feature: Notifications flow diagrams recreation #7470
Conversation
…e for first diagram
…e for email notification flow
…e for push notification flow
Hello @TomerPacific, Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @TomerPacific!
Newest code from TomerPacific has been published to preview environment for Git SHA 8280093 |
@lindy65 I would like to do the QA testing for this PR. Please assign this to me. Thanks. |
The changes in this PR have been verified and they strictly follow the guidelines mentioned in the original ticket.Here's a summary of the observed outcomes through this PR.
I've attached a video as a confirmation, comparing the existing flow diagrams and the newly created flow charts, and have also tried editing it successfully. PR-7470.QA.Testing.mp4cc: @lindy65 |
Summary
This PR recreates the following three diagrams using draw.io
All files are in a draw.io format and have been placed inside the images directory with an equivalent name to the same flow chart in .png format.
Ticket Link
Resolves #7348