-
Notifications
You must be signed in to change notification settings - Fork 554
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
Logic refactor & URL Schemes / 3D Touch #141
base: master
Are you sure you want to change the base?
Conversation
Better console log if already jailbroken
…d memory warning handler
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 LGTM
yalu102/Info.plist
Outdated
@@ -2,6 +2,26 @@ | |||
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd"> | |||
<plist version="1.0"> | |||
<dict> | |||
<key>UIApplicationShortcutItems </key> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Extra space in key name likely stops this from appearing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed, thanks b
^ We're currently using it to manipulate bools, and test 3D touch |
Okay, this should be good to merge now! |
@Mila432 What part of the code are you referring to? |
@nullpixel1 d02d694 no idea why GitHub moved the comment here |
@Mila432 ohhhh Look at the file it's in: it's my own jailbreak check, not the offsets. To shed more light on it, |
@nullpixel1 ye old code was c= a == b , new code c = a |
yeah, it was a debug statement @Mila432 |
@kpwn this PR now updates readme for new jailbreak, and should clarify some support questions. |
I am not OK with a URI scheme to initiate a jailbreak. Sounds sketch. |
@kpwn The user has to press ok to do it when launched via a URL scheme. It's mainly there for things such as workflow. |
@kpwn Sorry, forgot to commit the alert view, can you re-review this now? |
Rebased |
Resolved conflicts |
Bring up to base
Resolved merge conflict |
Bring up to date with master
Did you change your mind about URL schemes or should I factor them out? |
Bring up to date with base
didReceiveMemoryWarning
in offsets.c to correctly log the jailbroken state.
yalu://break
Bigger view controller refactor coming soon.