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

improvement(RNScreens): Add overrides for 2.0.0-beta.8 version which allows for package usage on native TV platforms #641

Merged
merged 1 commit into from
Jan 18, 2021

Conversation

RicardasN
Copy link
Collaborator

Description

In our project the current default version of react-native-screens (2.2.0) causes some unexpected issues, it works fine though with version 2.0.0-beta.8, however, some overrides are needed to get it working properly on TV platforms. Currently, as it stands RNScreens overrides are only applied to version 2.2.0 so this PR aims to introduce overrides for the version we use and perhaps someone else in the community uses, meaning they could avoid having to do the same overrides.

Breaking Changes

  • PRs should not introduce breaking changes to existing functionality
  • if breaking change cannot be avoided it has to be introduced in 2 phases (release cycles of 0.x.0)
    • 0.x.0 Add new functionality + add DEPRECATED warning to existing functionality
    • 0.[x+1].0 Remove deprecated functionality

I have tested my changes on:

ReNative project directly:

  • ios simulator
  • ios device
  • android simulator
  • android device
  • web browser
  • web -e next browser
  • tvos simulator
  • tvos device
  • androidtv simulator
  • androidtv device
  • androidwear simulator
  • androidwear device
  • tizen simulator
  • tizen device
  • tizenmobile simulator
  • tizenwatch device
  • webos simulator
  • webos device
  • macos
  • windows
  • chromecast device

New project:

  • ios simulator
  • ios device
  • android simulator
  • android device
  • web browser
  • web -e next browser
  • tvos simulator
  • tvos device
  • androidtv simulator
  • androidtv device
  • androidwear simulator
  • androidwear device
  • tizen simulator
  • tizen device
  • tizenmobile simulator
  • tizenwatch device
  • webos simulator
  • webos device
  • macos
  • windows
  • chromecast device

Existing Project created with previous version of renative:

  • ios simulator
  • ios device
  • android simulator
  • android device
  • web browser
  • web -e next browser
  • tvos simulator
  • tvos device
  • androidtv simulator
  • androidtv device
  • androidwear simulator
  • androidwear device
  • tizen simulator
  • tizen device
  • tizenmobile simulator
  • tizenwatch device
  • webos simulator
  • webos device
  • macos
  • windows
  • chromecast device

…allows for package usage on native TV platforms
@RicardasN RicardasN requested review from pavjacko and hosek January 15, 2021 08:58
@pavjacko pavjacko merged commit d640240 into develop Jan 18, 2021
@pavjacko pavjacko deleted the improvement/rn-screens-2.0.0-beta.8 branch February 15, 2022 09:08
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.

3 participants