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

Android Manifest Merge Issue (Unable to find explicit activity class) #228

Open
brassier opened this issue Nov 3, 2020 · 4 comments
Open
Assignees

Comments

@brassier
Copy link

brassier commented Nov 3, 2020

As noted in this ticket, there are often cases where the activity that is supposed to be added to the android manifest is not actually added. My hunch is that this happened due to this recent commit, for Android 10 support.

There are numerous folks claiming that edit-config has been known to remove chunks of the file being edited, which seems to be what happens here. The activity isn't reliably added to the manifest (i.e. it could be getting removed). Maybe there are other ways this can get the requestLegacyExternalStorage flag set? Like a hook, as suggested by the SO post below?

@brassier
Copy link
Author

brassier commented Nov 3, 2020

I tested out version 2.3.3, which is prior to the addition of the edit-config plugin.xml addition. I am seeing the <activity> element added to the manifest both on platform add (when the plugin has already been previously added), and on new plugin additions. This is more proof to me that this edit-config is causing problems with the manifest setup.

It would be great if this requestLegacyExternalStorage flag could move from the current edit-config to a hook. I think that would resolve the problem.

@anujraghuvanshi
Copy link

@brassier - Will be great & thankful if you could submit PR on your suggestions. We can take that up and add to next release soon.

@brassier
Copy link
Author

brassier commented Nov 6, 2020

@Anuj-Raghuvanshi - Yes, I should be able to get a PR out here on Monday. Thanks!

@brassier
Copy link
Author

brassier commented Nov 9, 2020

@Anuj-Raghuvanshi - PR is created (#232)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants