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

Remove support for Android API < 23 in ReactViewManager #39678

Closed
wants to merge 10 commits into from

Conversation

mdvacca
Copy link
Contributor

@mdvacca mdvacca commented Sep 26, 2023

Summary:
Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class ReactViewManager

chnagelog: [Android][Breaking] Remove support for Android API < 23 in ReactViewManager

Reviewed By: NickGerleman

Differential Revision: D48545513

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Sep 26, 2023
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48545513

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48545513

mdvacca added a commit to mdvacca/react-native that referenced this pull request Sep 26, 2023
Summary:
Pull Request resolved: facebook#39678

Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class ReactViewManager

chnagelog: [Android][Breaking] Remove support for Android API < 23 in ReactViewManager

Reviewed By: NickGerleman

Differential Revision: D48545513

fbshipit-source-id: 0b4dc462e5f16f6275f33f41feeb1d521d12d5be
@analysis-bot
Copy link

analysis-bot commented Sep 26, 2023

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 16,902,912 -6
android hermes armeabi-v7a n/a --
android hermes x86 n/a --
android hermes x86_64 n/a --
android jsc arm64-v8a 20,381,650 +3
android jsc armeabi-v7a n/a --
android jsc x86 n/a --
android jsc x86_64 n/a --

Base commit: 5be6f49
Branch: main

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48545513

mdvacca added a commit to mdvacca/react-native that referenced this pull request Sep 26, 2023
Summary:
Pull Request resolved: facebook#39678

Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class ReactViewManager

chnagelog: [Android][Breaking] Remove support for Android API < 23 in ReactViewManager

Reviewed By: NickGerleman

Differential Revision: D48545513

fbshipit-source-id: 0ef7a6fc681573f280b0ab8f5e7187bbd7c87802
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48545513

mdvacca added a commit to mdvacca/react-native that referenced this pull request Sep 26, 2023
Summary:
Pull Request resolved: facebook#39678

Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class ReactViewManager

chnagelog: [Android][Breaking] Remove support for Android API < 23 in ReactViewManager

Reviewed By: NickGerleman

Differential Revision: D48545513

fbshipit-source-id: b52975d9d0ef2a3766c41325326bc4a13ec4277a
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48545513

mdvacca added a commit to mdvacca/react-native that referenced this pull request Sep 26, 2023
Summary:
Pull Request resolved: facebook#39678

Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class ReactViewManager

chnagelog: [Android][Breaking] Remove support for Android API < 23 in ReactViewManager

Reviewed By: NickGerleman

Differential Revision: D48545513

fbshipit-source-id: 66325dbf226e4a98c0acd95f4f7a8346cb11d223
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48545513

mdvacca added a commit to mdvacca/react-native that referenced this pull request Sep 27, 2023
Summary:
Pull Request resolved: facebook#39678

Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class ReactViewManager

chnagelog: [Android][Breaking] Remove support for Android API < 23 in ReactViewManager

Reviewed By: NickGerleman

Differential Revision: D48545513

fbshipit-source-id: 8e28c452c43e79663f55e3ab543b422dc44a2ca2
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48545513

mdvacca added a commit to mdvacca/react-native that referenced this pull request Sep 27, 2023
Summary:
Pull Request resolved: facebook#39678

Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class ReactViewManager

chnagelog: [Android][Breaking] Remove support for Android API < 23 in ReactViewManager

Reviewed By: NickGerleman

Differential Revision: D48545513

fbshipit-source-id: 945b3805f72ea37555d9aca6b6d3dbc84d6e0207
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48545513

mdvacca added a commit to mdvacca/react-native that referenced this pull request Sep 28, 2023
Summary:
Pull Request resolved: facebook#39678

Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class ReactViewManager

changelog: [Android][Breaking] Remove support for Android API < 23 in ReactViewManager

Reviewed By: NickGerleman

Differential Revision: D48545513

fbshipit-source-id: 62ccca9e4b030666d0378fc9afca51e3e5c05817
@github-actions
Copy link

github-actions bot commented Sep 28, 2023

Warnings
⚠️ One hour and a half have passed and the E2E jobs haven't finished yet.

Generated by 🚫 dangerJS against 64ad64a

@popejrh19
Copy link

popejrh19 commented Sep 28, 2023 via email

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48545513

mdvacca added a commit to mdvacca/react-native that referenced this pull request Sep 28, 2023
Summary:
Pull Request resolved: facebook#39678

Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class ReactViewManager

changelog: [Android][Breaking] Remove support for Android API < 23 in ReactViewManager

Reviewed By: NickGerleman

Differential Revision: D48545513

fbshipit-source-id: fed98f9476cf95e107853cdaeb07f483ec87739c
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48545513

mdvacca added a commit to mdvacca/react-native that referenced this pull request Sep 28, 2023
Summary:
Pull Request resolved: facebook#39678

Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class ReactViewManager

changelog: [Android][Breaking] Remove support for Android API < 23 in ReactViewManager

Reviewed By: NickGerleman

Differential Revision: D48545513

fbshipit-source-id: 7c57a9ea850c0de012912a6454e8c282088e8b25
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48545513

mdvacca added a commit to mdvacca/react-native that referenced this pull request Sep 28, 2023
Summary:
Pull Request resolved: facebook#39678

Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class ReactViewManager

changelog: [Android][Breaking] Remove support for Android API < 23 in ReactViewManager

Reviewed By: NickGerleman

Differential Revision: D48545513

fbshipit-source-id: 1a20f14532d8bf12943987ff9a19596ef8d21066
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48545513

mdvacca added a commit to mdvacca/react-native that referenced this pull request Sep 29, 2023
Summary:
Pull Request resolved: facebook#39678

Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class ReactViewManager

changelog: [Android][Breaking] Remove support for Android API < 23 in ReactViewManager

Reviewed By: NickGerleman

Differential Revision: D48545513

fbshipit-source-id: 4050811b6026cce98764f6c82070bce4297e8201
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48545513

mdvacca added a commit to mdvacca/react-native that referenced this pull request Sep 29, 2023
Summary:
Pull Request resolved: facebook#39678

Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class ReactViewManager

changelog: [Android][Breaking] Remove support for Android API < 23 in ReactViewManager

Reviewed By: NickGerleman

Differential Revision: D48545513

fbshipit-source-id: 330d2e6784990c4c0f2526ac7981e710a08c068c
Summary:
Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class ReactTextShadowNode

chnagelog: [Android][Breaking] Remove support for Android API < 23 in ReactTextShadowNode

Differential Revision: D48545511

fbshipit-source-id: 68e887ea02193fbb0b6e706d0f868029ee7063fb
Summary:
Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class ReactTextView

chnagelog: [Android][Breaking] Remove support for Android API < 23 in ReactTextView

Differential Revision: D48545504

fbshipit-source-id: 769e0141365bdf1e98da7835afa3bf4ccff568f3
Summary:
Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class TextAttributeProps

chnagelog: [Android][Breaking] Remove support for Android API < 23 in TextAttributeProps

Differential Revision: D48545514

fbshipit-source-id: 366f3ebac9d77fd58afcf6060a0043a12c903bab
Summary:
Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class TextLayoutManager

chnagelog: [Android][Breaking] Remove support for Android API < 23 in TextLayoutManager

Differential Revision: D48545502

fbshipit-source-id: 6b70a9b5cf68efc5f9ebad6c7c1f78c1f6efb141
Summary:
Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class TextLayoutManagerMapBuffer

chnagelog: [Android][Breaking] Remove support for Android API < 23 in TextLayoutManagerMapBuffer

Differential Revision: D48545518

fbshipit-source-id: 7158be914604d0f8d271ff754a8a864be18e1fa9
Summary:
Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class ReactEditText

chnagelog: [Android][Breaking] Remove support for Android API < 23 in ReactEditText

Differential Revision: D48545507

fbshipit-source-id: da6e758420305119d30e73cfc0b7a44efe90ecc7
Summary:
Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class ReactTextInputLocalData

chnagelog: [Android][Breaking] Remove support for Android API < 23 in ReactTextInputLocalData

Differential Revision: D48545510

fbshipit-source-id: bd9004cf9f8b741ff0655555b8a9afa3f66f1594
Summary:
Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class ReactTextInputShadowNode

chnagelog: [Android][Breaking] Remove support for Android API < 23 in ReactTextInputShadowNode

Differential Revision: D48545503

fbshipit-source-id: 6ee922970c9983a2fba27028df02a3c63700d996
Summary:
Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class ReactDrawableHelper

chnagelog: [Android][Breaking] Remove support for Android API < 23 in ReactDrawableHelper

Differential Revision: D48545508

fbshipit-source-id: ca98c4dae65101dca78ac78bd5af7950fe5167a9
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48545513

Summary:
Pull Request resolved: facebook#39678

Since minsdk version was increased to 23, we are deleting code using Android APIs < 23 for class ReactViewManager

changelog: [Android][Breaking] Remove support for Android API < 23 in ReactViewManager

Reviewed By: NickGerleman

Differential Revision: D48545513

fbshipit-source-id: 792826fd44e756afcd4659a95212e82f256525e3
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Sep 29, 2023
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in cb46f4c.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants