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

refactor(example): re-init example #3470

Closed
wants to merge 6 commits into from
Closed

refactor(example): re-init example #3470

wants to merge 6 commits into from

Conversation

KrzysztofMoch
Copy link
Member

@KrzysztofMoch KrzysztofMoch commented Jan 12, 2024

Summary

To clean code and catch up with newest template I have re-created basic example

NOTE: I remove windows from basic example because it's very heavy - should (will) be moved to separate example

fixes #3469

Changes (PR change log is little mess... 😅)

  • re-inited basic example
  • removed FabricExample & embed-and-fullscreen examples (basic should be used to test fabric)

Test Plan

  • App build and start with:
    1. yarn install
    2. yarn build
    3. yarn xbasic install
    4. yarn xbasic android
  • CI pass

@freeboub

This comment was marked as off-topic.

@KrzysztofMoch

This comment was marked as off-topic.

@freeboub
Copy link
Collaborator

freeboub commented Feb 1, 2024

@KrzysztofMoch When building from scratch I still have the build issue:

It is really not clear why I have this issue when cloning from scratch, but not on my previously cloned repo :/

transform[stderr]: Could not resolve "/Users/olivier/Documents/GitHub/react-native-video/src/index" in file /Users/olivier/Documents/GitHub/react-native-video/examples/basic/src/VideoPlayer.tsx.
transform[stderr]: Could not resolve "/Users/olivier/Documents/GitHub/react-native-video/src/index" in file /Users/olivier/Documents/GitHub/react-native-video/examples/basic/src/MultiValueControl.tsx.
BUNDLE src/index.js

error: Error: Unable to resolve module @babel/runtime/helpers/interopRequireDefault from /Users/olivier/Documents/GitHub/react-native-video/src/index.ts: @babel/runtime/helpers/interopRequireDefault could not be found within the project or in these directories:

Note: I also think you broken windows sample... it will need a reallignment in the future

@@ -1,23 +1,13 @@
<manifest xmlns:android="http://schemas.android.com/apk/res/android">

<uses-feature
Copy link
Collaborator

Choose a reason for hiding this comment

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

can you please revert changes in this file to keep "current" android tv support ?

"private": true,
"scripts": {
"android": "react-native run-android",
"ios": "react-native run-ios",
"windows": "react-native run-windows",
Copy link
Collaborator

Choose a reason for hiding this comment

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

keep windows please

},
"dependencies": {
"@react-native-picker/picker": "^1.9.11",
"@react-native-picker/picker": "react-native-picker/picker",
Copy link
Collaborator

Choose a reason for hiding this comment

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

strange, what does it do ?

Copy link
Member Author

Choose a reason for hiding this comment

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

I really don't know 😅 i will remove it ...

"react": "18.2.0",
"react-native": "0.73.2",
"react-native-windows": "0.63.41"
Copy link
Collaborator

Choose a reason for hiding this comment

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

keep react-native-windows please

@KrzysztofMoch
Copy link
Member Author

I will close this one as there is a lot of conflicts and current examples seems to work again

@KrzysztofMoch KrzysztofMoch deleted the refactor/reinit-example branch March 13, 2024 07:31
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.

Example build failed on android
2 participants