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 0.60 tasks #1172

Closed
37 of 44 tasks
necolas opened this issue Nov 12, 2018 · 44 comments
Closed
37 of 44 tasks

React Native 0.60 tasks #1172

necolas opened this issue Nov 12, 2018 · 44 comments
Assignees
Milestone

Comments

@necolas
Copy link
Owner

necolas commented Nov 12, 2018

Tasks related to changes in React Native 0.60.

Afterwards:

  • Add accessibilitySetSize
  • Add accessibilityPosInSet
  • Add direction (inc none)
  • Remove accessible
  • Add focusable
  • Remove enzyme
@jstheoriginal
Copy link

Are the update snasphot tasks simply taking the appropriate folder from RN 0.57 and replacing the folder in this project? If not, do you have the steps required for that type of task?

Also, should we work off of RN 0.57.4 tag? https://github.com/facebook/react-native/tree/v0.57.4

@necolas
Copy link
Owner Author

necolas commented Nov 13, 2018

Updated the OP. RN 0.57.5. Snapshots are of files, not directories. You'll also need to handle updating any new import paths or dependencies

necolas added a commit that referenced this issue Nov 17, 2018
necolas added a commit that referenced this issue Nov 17, 2018
Mirror contents of React Native 0.57.5

Ref #1172
necolas added a commit that referenced this issue Nov 17, 2018
Mirror contents of React Native 0.57.5

Ref #1172
necolas added a commit that referenced this issue Nov 17, 2018
Mirror contents of React Native 0.57.5

Ref #1172
necolas added a commit that referenced this issue Nov 17, 2018
Mirror contents of React Native 0.57.5

Ref #1172
necolas added a commit that referenced this issue Nov 17, 2018
Mirror contents of React Native 0.57.5

Ref #1172
necolas added a commit that referenced this issue Nov 17, 2018
Mirror contents of React Native 0.57.5

Ref #1172
@necolas necolas mentioned this issue Dec 20, 2018
@unindented
Copy link

I'll try to take a look at Replace context.isInAParentText w/ React.createContext, see if it's too big for me.

@necolas
Copy link
Owner Author

necolas commented Dec 21, 2018

I accidentally wiped out a bunch of commits on the react-native-0.57 branch because I forgot to pull down changes I pushed from another computer. Need to fix that when I'm back in SF

necolas added a commit that referenced this issue Dec 30, 2018
necolas added a commit that referenced this issue Dec 30, 2018
Mirror contents of React Native 0.57.5

Ref #1172
necolas added a commit that referenced this issue Dec 30, 2018
Mirror contents of React Native 0.57.5

Ref #1172
necolas added a commit that referenced this issue Dec 30, 2018
Mirror contents of React Native 0.57.5

Ref #1172
necolas added a commit that referenced this issue Dec 30, 2018
Mirror contents of React Native 0.57.5

Ref #1172
necolas added a commit that referenced this issue Dec 30, 2018
Mirror contents of React Native 0.57.5

Ref #1172
necolas added a commit that referenced this issue Dec 30, 2018
Mirror contents of React Native 0.57.5

Ref #1172
necolas added a commit that referenced this issue Dec 30, 2018
Mirror contents of React Native 0.57.5

Ref #1172
necolas added a commit that referenced this issue Dec 31, 2018
Mirror contents of React Native 0.57.5

Ref #1172
necolas added a commit that referenced this issue Dec 31, 2018
Mirror contents of React Native 0.57.5

Ref #1172
necolas added a commit that referenced this issue Dec 31, 2018
Mirror contents of React Native 0.57.5

Ref #1172
necolas added a commit that referenced this issue Dec 31, 2018
Mirror contents of React Native 0.57.5

Ref #1172
@evelant
Copy link

evelant commented Jul 1, 2019

I'm also curious about this issue. Won't apps using < 0.59 start being rejected from app stores starting in August because of 64bit? After that point it will more or less be impossible to use an old version like 0.55.

@asgvard
Copy link

asgvard commented Jul 1, 2019

Latest RNW works fine with RN 0.59, just officially not supported.
However would be appreciated to have any guidance to finish these tasks, I guess ppl just need one clear example on what does it mean to update snapshots and how to do it :)

@piranna
Copy link

piranna commented Jul 1, 2019

Latest RNW works fine with RN 0.59, just officially not supported.

What does it needs to get to be officially supported (besides an update in README.md :-) )?

@necolas necolas changed the title React Native 0.57 tasks React Native 0.60 tasks Jul 2, 2019
@necolas necolas self-assigned this Jul 2, 2019
@tastafur
Copy link

tastafur commented Aug 9, 2019

Hello,
I would like to make these three tasks @necolas .
Update RNTester snapshot
Add forwardedRef prop to Text
Add forwardedRef prop to View
Do I open an issue with your Pull Request for each task?

@tastafur
Copy link

tastafur commented Aug 9, 2019

Hello,
@necolas In this task Update RNTester snapshot, what you want to do is add test snapshot to all RNTester components?

necolas added a commit that referenced this issue Aug 9, 2019
Replaces the legacy 'contextTypes' API.
Removes 'enzyme' from related tests. Back to recording DOM snapshots.

Close #1248
Ref #1172
necolas added a commit that referenced this issue Aug 9, 2019
Adds the accessibilityState and accessibilityRelationship object props that map
to ARIA props.

Removes the accessibilityStates array prop that is not compatible with web
accessibility services.

Ref #1172
necolas added a commit that referenced this issue Aug 16, 2019
Replaces the legacy 'contextTypes' API.
Removes 'enzyme' from related tests. Back to recording DOM snapshots.

Close #1248
Ref #1172
necolas added a commit that referenced this issue Aug 16, 2019
Adds the accessibilityState and accessibilityRelationship object props that map
to ARIA props.

Removes the accessibilityStates array prop that is not compatible with web
accessibility services.

Ref #1172
@prade
Copy link

prade commented Aug 29, 2019

I have RNW working with RN 0.60 after doing multiple config overrides using webpack and babel, but unable to get a production build. Waiting for official support so we can override the hacks.
Thanks for the library @necolas , this definitely is the future for react development.

Latest RNW works fine with RN 0.59, just officially not supported.
However would be appreciated to have any guidance to finish these tasks, I guess ppl just need one clear example on what does it mean to update snapshots and how to do it :)

@piranna
Copy link

piranna commented Aug 29, 2019

@prade can you be able to publish your hacks in a pull-request? This way they could be easier merged in master.

@piranna
Copy link

piranna commented Aug 29, 2019

And not only that, that also they could be used by others.

@tastafur
Copy link

Hello
I would like to contribute to these tasks that I have to do? @necolas

necolas added a commit that referenced this issue Oct 7, 2019
Replaces the legacy 'contextTypes' API.
Removes 'enzyme' from related tests. Back to recording DOM snapshots.

Close #1248
Ref #1172
necolas added a commit that referenced this issue Oct 7, 2019
Adds the accessibilityState and accessibilityRelationship object props that map
to ARIA props.

Removes the accessibilityStates array prop that is not compatible with web
accessibility services.

Ref #1172
necolas added a commit that referenced this issue Oct 7, 2019
necolas added a commit that referenced this issue Oct 7, 2019
Reference RN 0.60 implementations with web-specific fixes

Ref #1172
Fix #1105
@necolas
Copy link
Owner Author

necolas commented Oct 7, 2019

Decided not to add more onPress props to Text for the reasons given here #1272 (comment)

I've been thinking about this feature for a while, and I think I'm going to exclude it from web by design. Apps like Twitter are using Text everywhere and adding the build and runtime overhead of Touchable to Text is (on balance) going to make the most common use of Text become more expensive. I think RN is doing too much in Text (it didn't use to overload it with press functionality) vs View, and RN should probably redesign its exports to include something like TouchableText.

@necolas
Copy link
Owner Author

necolas commented Oct 7, 2019

Updating babel-plugin-tester is currently blocked on babel-utils/babel-plugin-tester#50.

Edit: babel-plugin-tester's unit tests didn't catch the change, and the author closed the issue. I will stick with v5 and stop using babel-plugin-tester in the future. Alternatives: https://babeljs.io/docs/en/next/babel-helper-plugin-test-runner.html, https://github.com/satya164/babel-test

@kopax
Copy link

kopax commented Oct 28, 2019

Helo, I am just discovering react-native and I wonder if I should use it for the cross platforme native and web app I want to build.

I see that 0.60 of react-native is out but this is blocked for a while and there's no milestone or release date for this. Can I have you people users opinions, should I expect to be able to upgrade to 0.60 without pain the future if I start building my app with react-native-web or should I just not use react-native-web ?

Thanks for sharing !

@mykhailo-melnyk
Copy link

@kopax
I am also interested in this

@kopax
Copy link

kopax commented Nov 11, 2019

I still do not have my answer and so far I have expo, and making my own. I have not decided yet. Not sure if this should be use, but shouldn't be too hard to make your own react-native-web I am also thinking this as a solution.

@necolas
Copy link
Owner Author

necolas commented Nov 12, 2019

I don't know what you're asking. Nothing is blocked on anything and Expo uses RNW with RN 0.60. I'm going to close comments on this issue because they're not helpful and most of the work is complete and will land in master in good time

Repository owner locked and limited conversation to collaborators Nov 12, 2019
@necolas necolas closed this as completed Nov 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests