-
Notifications
You must be signed in to change notification settings - Fork 406
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
add test cases for note images #420
add test cases for note images #420
Conversation
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 for working on this!
I would like to improve some of the locators and other things if possible.
Could you rebase your changes on |
c700738
to
da00327
Compare
Thanks again! Could you revert changes to the package lock? I don't think that file should have changes |
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!
909a8b2
to
c934acd
Compare
Adding tests for note images.
The following tests are included:
Test Plan
Checklist
Screenshots