Skip to content
This repository has been archived by the owner on May 8, 2024. It is now read-only.

Upgrade to 0.60.4 #26

Merged
merged 13 commits into from
Aug 6, 2019
Merged

Upgrade to 0.60.4 #26

merged 13 commits into from
Aug 6, 2019

Conversation

wswebcreation
Copy link
Contributor

Sorry for the big ass PR, but this one was needed for the following reasons:

  • we needed to upgrade to the latest version of React Native to support the latest version of Android
  • React Native made some really big changes to the source code (that's why we have that amount of files touched, please focus only on the tests and src folders )
  • We needed to update a lot of modules due to changes in them
  • a lot of customers are asking on how to automate deeplinking and biometrical login and I can only integrate that when I updated the project

So, basically, everything changed 😆

I'm on holiday as of now, so take your time. When this is merged I will make new releases for the Sauce Labs users and customers and will upload it here https://github.com/saucelabs/sample-app-mobile/releases

That was it, happy checking!

- upgrade test dependencies
- add visual regression testing including script
- add images
- clean up configs
- upgrade for iOS, only for login screen
- upgrade for Android
- fix Android splash screen
- fix Android icons
- fix Android fonts
- add new webview component
- fix linting
- fix Android release build
- fix linting by adding a hook
- fix Webview for Android
- fix layout for all components
- add a close button translation
- add Appium to the services
- fix iOS config
- add fix for Android accessibilityLabels, due to an issue on React Native
- fix tests
- fix readme
- fix Android fonts
- fix visual tests
Copy link
Contributor

@rslobojan-saucelabs rslobojan-saucelabs left a comment

Choose a reason for hiding this comment

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

Deleting the LICENSE as part of this PR would be A Bad Thing[tm] - please don't do that. Also have a few questions in the review comments.

Other than that, changeset looks pretty good - thanks for putting this together!

LICENSE Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
package-lock.json Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@wswebcreation
Copy link
Contributor Author

wswebcreation commented Aug 4, 2019

@rslobojan-saucelabs

As said, thanks for taking the time to review it. I fixed the changes and updated the comment, can you please take a look and if you agree resolve the last remark?
I'll then merge it this week and create a new release

@wswebcreation wswebcreation merged commit 2b58d4c into master Aug 6, 2019
@wswebcreation wswebcreation deleted the feat/upgrade-rn-0.60.4 branch August 6, 2019 06:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants