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

React Native Windows >=0.63.11 Initial Implementation PR #1994

Merged
merged 9 commits into from
Dec 16, 2020

Conversation

alexmoroz15
Copy link
Contributor

Adds react-native-windows >= 0.63.11 functionality to react-native-code-push.

Updated windows setup instructions and provided a demo app under Examples/ called CodePushDemoAppCpp.

Windows implementation has not been configured to work with the testing suite.

I have not checked to see whether these changes break the ios and android implementations.

The following features are missing and still need to be implemented:

  • The ability to recover consistently from a failed update; a corrupted JS bundle might prevent the app from working.
  • ON_NEXT_RESUME and ON_NEXT_SUSPEND install modes are not currently supported
  • JWT signature verification is not currently supported
  • Update content verification is not currently supported (for example, if an update is a diff update) - I was unable to recreate the hashing algorithm used by CodePush.

{
"name": "CodePushDemoAppCpp",
"displayName": "CodePushDemoAppCpp"
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you please fix No newline at the end of file warning in all files?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have added newlines at the end of each file. Hopefully the warnings are fixed now.

@vmoroz
Copy link
Member

vmoroz commented Dec 15, 2020

@andreidubov, @alexandergoncharov, the CI fails with the iOS test. I believe this change must not affect any iOS or Android behavior and the failures do no seem to be related. Is it OK to merge the PR and ignore these failures?

@vmoroz vmoroz merged commit 75785cb into microsoft:master Dec 16, 2020
sourabhdebnath pushed a commit to scripbox/react-native-code-push that referenced this pull request May 2, 2022
)

* Add React Native Windows 0.63.5+ support

* modify windows/.npmignore

* Update setup-windows.md

* Revert package.json changes

* Add newline at the end of added files

* Update demo app package.json to show minimum version

* Update setup-windows.md

* Update setup-windows.md

Co-authored-by: Aleksey <[email protected]>
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.

4 participants