Skip to content

Fix @react-native/popup-menu-android not building for 3rd party developers#49212

Closed
cortinico wants to merge 1 commit intofacebook:mainfrom
cortinico:export-D69192874
Closed

Fix @react-native/popup-menu-android not building for 3rd party developers#49212
cortinico wants to merge 1 commit intofacebook:mainfrom
cortinico:export-D69192874

Conversation

@cortinico
Copy link
Contributor

Summary:
Currently, developers can't use popup-menu-android at all because the Gradle file we publish is referencing
internal machinery.

I'm adding a pre-publish script that manipulates the Gradle. This is the easiest solution without having to do
crazy setup inside RNGP or having duplicated version codes around in the monorepo.

Fixes #49112

Changelog:
[Android] [Fixed] - Fix react-native/popup-menu-android not building for 3rd party developers

Differential Revision: D69192874

@facebook-github-bot facebook-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 Feb 5, 2025
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

…opers (facebook#49212)

Summary:

Currently, developers can't use `popup-menu-android` at all because the Gradle file we publish is referencing
internal machinery.

I'm adding a pre-publish script that manipulates the Gradle. This is the easiest solution without having to do
crazy setup inside RNGP or having duplicated version codes around in the monorepo.

Fixes facebook#49112

Changelog:
[Android] [Fixed] - Fix react-native/popup-menu-android not building for 3rd party developers

Reviewed By: cipolleschi

Differential Revision: D69192874
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in e96396b.

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Feb 6, 2025
@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @cortinico in e96396b

When will my fix make it into a release? | How to file a pick request?

@gustavoabel
Copy link
Contributor

gustavoabel commented Feb 6, 2025

@cortinico Hey Cortinico! Is this package ready to be used? When importing it into my project, it is not recognized in RN 0.76

@cortinico
Copy link
Contributor Author

@cortinico Hey Cortinico! Is this package ready to be used? When importing it into my project, it is not recognized in RN 0.76

The fix is in main. It will have to be cherry-picked in the release branches of the version you're using: https://github.com/reactwg/react-native-releases/#what-is-a-pick-request

@gustavoabel
Copy link
Contributor

@cortinico Hey Cortinico! Is this package ready to be used? When importing it into my project, it is not recognized in RN 0.76

The fix is in main. It will have to be cherry-picked in the release branches of the version you're using: reactwg/react-native-releases#what-is-a-pick-request

Thanks and done! reactwg/react-native-releases#803

react-native-bot pushed a commit that referenced this pull request Mar 11, 2025
…opers (#49212)

Summary:
Pull Request resolved: #49212

Currently, developers can't use `popup-menu-android` at all because the Gradle file we publish is referencing
internal machinery.

I'm adding a pre-publish script that manipulates the Gradle. This is the easiest solution without having to do
crazy setup inside RNGP or having duplicated version codes around in the monorepo.

Fixes #49112

Changelog:
[Android] [Fixed] - Fix react-native/popup-menu-android not building for 3rd party developers

Reviewed By: cipolleschi

Differential Revision: D69192874

fbshipit-source-id: 9f9e8a0a6e76308e598a09f4c70dbc659c238b00
@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @cortinico in ac637ff

When will my fix make it into a release? | How to file a pick request?

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.

@react-native/popup-menu-android causes build failure on android: build.gradle.kts Unresolved reference: libs

4 participants