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

Add rewards test #140

Merged
merged 2 commits into from Oct 21, 2019
Merged
Changes from 1 commit
Commits
File filter...
Filter file types
Jump to…
Jump to file
Failed to load files.

Always

Just for now

Prev

Fix review comments

  • Loading branch information
srirambv committed Oct 19, 2019
commit 2982defaee04e23f0999e58bfc56b2c226cf865e
@@ -17,7 +17,7 @@ Pre-Requisite: Put previous build shortcut on the home screen. Also, have severa
- [ ] Verify shortcut is still available on the home screen after upgrade
- [ ] Verify sites added to home screen are still visible and able to be used after upgrade
- [ ] Verify sync chain created in the previous version is still retained on upgrade
- [ ] Verify settings changes done in the previous version is still retained on upgrade
- [ ] Verify settings changes done in the previous version are still retained on upgrade

## Bookmarks

@@ -142,15 +142,15 @@ Pre-Requisite: Put previous build shortcut on the home screen. Also, have severa

## Sync
This conversation was marked as resolved by srirambv

This comment has been minimized.

Copy link
@LaurenWags

LaurenWags Aug 29, 2019

Contributor

general wording in Sync section - noticed you primarily used 'are able' instead of 'can' in iOS, consider changing here as well for consistency.


- [ ] Verify you can join sync chain by scanning the QR code
- [ ] Verify you can join sync chain using code words
- [ ] Verify you can create a sync chain on the device and add other devices to the chain via QR code/Codewords
- [ ] Verify you are able to join sync chain by scanning the QR code
- [ ] Verify you are able to join sync chain using code words
- [ ] Verify you are able to create a sync chain on the device and add other devices to the chain via QR code/Code words
- [ ] Verify that bookmarks from other devices on the chain show up on the mobile device after sync completes
- [ ] Verify newly created bookmarks gets sync'd to all devices on the sync chain
This conversation was marked as resolved by srirambv

This comment has been minimized.

Copy link
@LaurenWags

LaurenWags Aug 29, 2019

Contributor

Line 149 - in iOS you had changed sync'd to synced - probably should do it here for consistency.

This comment has been minimized.

Copy link
@srirambv

srirambv Oct 19, 2019

Author Contributor

Will update on iOS. It needs to be sync'd everywhere.

- [ ] Verify existing bookmarks before joining sync chain also gets sync'd to all devices on the sync chain
This conversation was marked as resolved by srirambv

This comment has been minimized.

Copy link
@LaurenWags

LaurenWags Aug 29, 2019

Contributor

if you change sync'd above, change here too.

- [ ] Verify sync works on an upgrade profile and new bookmarks added post-upgrade sync's across devices on the chain
- [ ] Verify add a bookmark on custom tab gets synced across all devices in the chain
- [ ] Verify you can create a standalone sync chain with one device
- [ ] Verify adding a bookmark on custom tab gets synced across all devices in the chain
- [ ] Verify you are able to create a standalone sync chain with one device

## Top sites view

ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.