Skip to content

feat: Add setting menu to save patched app crash logs to file#143

Open
prateek-who wants to merge 2 commits intoMorpheApp:devfrom
prateek-who:adb-logs
Open

feat: Add setting menu to save patched app crash logs to file#143
prateek-who wants to merge 2 commits intoMorpheApp:devfrom
prateek-who:adb-logs

Conversation

@prateek-who
Copy link
Copy Markdown
Contributor

No description provided.

@LisoUseInAIKyrios
Copy link
Copy Markdown
Contributor

Crashes are captured perfectly.

Intentional crash from a bad patching:

AndroidRuntime: Shutting down VM
AndroidRuntime: FATAL EXCEPTION: main
AndroidRuntime: Process: app.morphe.android.youtube, PID: 29880
AndroidRuntime: java.lang.NoSuchMethodError: No static method nonExistentMethod(Landroid/view/ViewGroup;)V in class Lapp/morphe/extension/youtube/sponsorblock/ui/SponsorBlockViewController; or its super classes (declaration of 'app.morphe.extension.youtube.sponsorblock.ui.SponsorBlockViewController' appears in /data/app/~~wgcBihf-YcwObm9foAP_eg==/app.morphe.android.youtube-z7WFxVvyo49zHwLLClRClA==/base.apk!classes2.dex)
AndroidRuntime: 	at nam.G(PG:110)
AndroidRuntime: 	at nam.z(PG:1)

@LisoUseInAIKyrios LisoUseInAIKyrios changed the title feat: abd logs are now captured and logged feat: Add setting menu to save patched app crash logs to file May 9, 2026
@prateek-who
Copy link
Copy Markdown
Contributor Author

Is the UI fine for this? Or does it feel confusing or not apparent enough?

@LisoUseInAIKyrios
Copy link
Copy Markdown
Contributor

It works how it is, but it's not a setting.

Maybe a second menu can exist for tools. A second button beside settings or something.

@LisoUseInAIKyrios
Copy link
Copy Markdown
Contributor

Unrelated to this, but the patch source title on the main screen could be clickable that opens a drop down menu or a dialog to change the patch source.

@prateek-who
Copy link
Copy Markdown
Contributor Author

Yup, working on that right now. Manager does it much better by listing all patchable apps once a source is added, so implementing that first in the gui. Will have to change the cli so that it allows it too after this.

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

Successfully merging this pull request may close these issues.

2 participants