Skip to content

feat: read main activity from AndroidManifest.xml automatically #1967

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.

Sign up for GitHub

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

Jump to bottom
Merged
thymikee merged 6 commits into from
Aug 17, 2023
Merged

feat: read main activity from AndroidManifest.xml automatically #1967

thymikee merged 6 commits into from
Aug 17, 2023

Conversation

@szymonrybczak
Copy link
Collaborator

@szymonrybczak szymonrybczak commented Jun 14, 2023 •
edited
Loading

Summary:

Fixes: #896

In this PR I created getMainActivity function that reads AndroidManifest.xml file and gets the main activity name.
Before this PR when user changed the main activity they have to manually provide main activity by option --main-activity.

Test Plan:

  1. Clone the repository and do all the required steps from the Contributing guide
  2. Change main activity name.
  3. Run this command:
node /path/to/react-native-cli/packages/cli/build/bin.js run-android

Main activity should be correctly read from AndroidManifest.xml and correctly launched on emulator.

@github-actions github-actions bot added docs Documentation change feature labels Jun 14, 2023
@szymonrybczak szymonrybczak force-pushed the feat/readMainActivityAutomatically branch 2 times, most recently from 3113c8b to bc0ca64 Compare June 14, 2023 09:15
@szymonrybczak szymonrybczak force-pushed the feat/readMainActivityAutomatically branch from bc0ca64 to e16ff7e Compare June 14, 2023 10:55
@szymonrybczak szymonrybczak marked this pull request as ready for review June 14, 2023 11:16
@szymonrybczak
Copy link
Collaborator Author

szymonrybczak commented Jun 14, 2023

@cortinico Would you mind looking at this please? 🙏

TMisiukiewicz
TMisiukiewicz reviewed Jul 13, 2023
View reviewed changes
@thymikee thymikee merged commit 7342221 into react-native-community:main Aug 17, 2023
@szymonrybczak szymonrybczak deleted the feat/readMainActivityAutomatically branch August 17, 2023 13:06
@WoLewicki WoLewicki mentioned this pull request Nov 23, 2023
Merged
2 tasks
@cameroncharles cameroncharles mentioned this pull request May 16, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cortinico cortinico cortinico approved these changes

@thymikee thymikee thymikee approved these changes

@TMisiukiewicz TMisiukiewicz TMisiukiewicz approved these changes

@adamTrz adamTrz Awaiting requested review from adamTrz

Assignees

No one assigned

Labels

docs Documentation change feature

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

auto detect mainActivity from AndroidManifest.xml

4 participants

@szymonrybczak @cortinico @thymikee @TMisiukiewicz