Skip to content

Remove some methods that have NS_UNAVAILABLE set - #19790

Closed
pvinis wants to merge 1 commit into
react:masterfrom
pvinis:fix/method-implementation-defines
Closed

Remove some methods that have NS_UNAVAILABLE set#19790
pvinis wants to merge 1 commit into
react:masterfrom
pvinis:fix/method-implementation-defines

Conversation

@pvinis

Copy link
Copy Markdown
Contributor

These lines were causing erros with the e2e tests on ios.
There are some methods that are not going to be called, and some definitions that are not correct.

needed for #19574.

Test Plan

we will run the e2e tests.

Related PRs

#19574

Release Notes

[INTERNAL] [MINOR] [Tests] - Fix some definitions.

@facebook-github-botfacebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 18, 2018
@pvinispvinis changed the title no need for these because they have NS_UNAVAILABLE setRemove some methods that have NS_UNAVAILABLE setJun 18, 2018
@pvinispvinis mentioned this pull request Jun 18, 2018
@facebook-github-botfacebook-github-bot added the Import Started This pull request has been imported. This does not imply the PR has been approved. label Jun 19, 2018

@facebook-github-botfacebook-github-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@hramos is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

sidnair pushed a commit to taptapsend/react-native that referenced this pull request Aug 2, 2018
Summary:
These lines were causing erros with the e2e tests on ios.
There are some methods that are not going to be called, and some definitions that are not correct.
needed for react#19574.
we will run the e2e tests.
[INTERNAL] [MINOR] [Tests] - Fix some definitions.
Closesreact#19790
Differential Revision: D8509228
Pulled By: hramos
fbshipit-source-id: 78650e347db52305a587e3461ac56877e3c79062
sidnair pushed a commit to taptapsend/react-native that referenced this pull request Aug 21, 2018
Summary:
These lines were causing erros with the e2e tests on ios.
There are some methods that are not going to be called, and some definitions that are not correct.
needed for react#19574.
we will run the e2e tests.
[INTERNAL] [MINOR] [Tests] - Fix some definitions.
Closesreact#19790
Differential Revision: D8509228
Pulled By: hramos
fbshipit-source-id: 78650e347db52305a587e3461ac56877e3c79062
@hramoshramos added Merged This PR has been merged. and removed Import Started This pull request has been imported. This does not imply the PR has been approved. labels Mar 8, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.MergedThis PR has been merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@pvinis@facebook-github-bot@hramos@react-native-bot