-
Notifications
You must be signed in to change notification settings - Fork 4
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
fix(previews): add checks to create payment preview messages #664
Conversation
Download the app installers for this pull request: |
Automated Test ResultsDetails 125 tests across 16 suites Skipped testsAutomated Tests on Chrome Desktop › 03-friends-two-instances.spec.ts › Two instances tests - Friends and Chats › B8 to B14 - Quick Profile Local and Remote - Updating note |
Playwright test execution is complete! You can find the test results report 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.
I requested a change to let code more clear.
What this PR does 📖
Which issue(s) this PR fixes 🔨
Special notes for reviewers 🗒️
Big thing I was looking to do is have the messages apply correctly for who sent, and who is receiving and using the correct grammar depending on who sent/received
Additional comments 🎤