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

doc: add kotlin MainActiviy.kt to instructions #1971

Merged
merged 3 commits into from
Nov 16, 2023

Conversation

mfazekas
Copy link
Contributor

Description

RN 0.73 has MainActivity in kotlin not java

Changes

  • Updated README.md

kkafar and others added 3 commits November 6, 2023 09:44
…sion#1957)

## Description

Setting version of JDK to 17 for CI builds as it is required by 0.73 to
run.

## Test code and steps to reproduce

CI passing should be enough

## Checklist

- [ ] Ensured that CI passes
software-mansion#1969)

## Description

Currently, in iOS 17.2 Beta 1-2 there's a bug that causes application to
crash after going to the second screen with the video player that
contains controls. This PR fixes this issue.

Closes software-mansion#1967.

## Changes

- Added an if statement that returns if view is type of `AVPlayerView`,
as we don't want to signal it about decelerating (sadly, this header
file is not in UIKit's public API 😕.

## Screenshots / GIFs

### Before


https://github.com/software-mansion/react-native-screens/assets/23281839/6d040536-45fb-41ea-ad8a-1c86352bf199

### After


https://github.com/software-mansion/react-native-screens/assets/23281839/6da618e4-2bf9-47fb-be9f-e73ae560f081

## Test code and steps to reproduce

You can test this change on the repro, attached in the original issue:
https://github.com/abanobmikaeel/react-native-video-crash/

## Checklist

- [X] Ensured that CI passes
@kkafar
Copy link
Member

kkafar commented Nov 16, 2023

Look great, thank you for contributing. I'll merge it to #1956 where we work on support for 0.73

@kkafar kkafar changed the base branch from main to @kkafar/rn-73 November 16, 2023 09:22
@kkafar kkafar merged commit 4a44260 into software-mansion:@kkafar/rn-73 Nov 16, 2023
1 check passed
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