Skip to content

covered issue-718 - added spec for search recipient by name and select from list - #797

Merged
tomholub merged 7 commits into
masterfrom
tests/issue-718
Oct 29, 2021
Merged

tomholub merged 7 commits into
masterfrom
tests/issue-718

Conversation

@fcvakintos

@fcvakintos fcvakintos commented Oct 26, 2021

Copy link
Copy Markdown
Contributor

This PR covers issue #718

close #718

Tests (delete all except exactly one):

  • Tests added or updated

@fcvakintos fcvakintos changed the title covered issue-718 covered issue-718 - search and choose a recipient in compose email Oct 26, 2021
@fcvakintos fcvakintos changed the title covered issue-718 - search and choose a recipient in compose email covered issue-718 - added spec for search recipient by name and select from list Oct 26, 2021

@tomholub tomholub left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please see comments

Comment thread appium/tests/screenobjects/new-message.screen.ts Outdated
Comment thread appium/tests/screenobjects/new-message.screen.ts Outdated
Comment thread appium/tests/screenobjects/new-message.screen.ts
@tomholub
tomholub marked this pull request as draft October 27, 2021 07:12
Comment thread appium/tests/data/index.ts
@tomholub

tomholub commented Oct 29, 2021

Copy link
Copy Markdown
Collaborator

@fcvakintos if you skip CI on last commit of the PR, then I won't know if it passes in CI or not, and therefore can't merge.

// edit: also need to mark the PR as ready for review

@tomholub

Copy link
Copy Markdown
Collaborator

image

@fcvakintos
fcvakintos marked this pull request as ready for review October 29, 2021 14:34
@tomholub
tomholub merged commit 0e35e37 into master Oct 29, 2021
@tomholub
tomholub deleted the tests/issue-718 branch October 29, 2021 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ui test to search and choose a recipient (no need to send)

3 participants