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

automatically add whitelist from activation scope #179

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rhjdvsgsgks
Copy link

maybe a little bit helpful when using whitelist and activation scope. no longer needs to select application twice(once in whitelist, once in activation scope) anymore

@MlgmXyysd
Copy link
Member

MlgmXyysd commented Feb 7, 2021

Then why not use LSPosed

So why force to open whitelist and scope to disgust users

@ElderDrivers ElderDrivers deleted a comment from yujincheng08 Feb 7, 2021
@ElderDrivers ElderDrivers deleted a comment from kotori2 Feb 7, 2021
@ElderDrivers ElderDrivers deleted a comment from yujincheng08 Feb 7, 2021
@rhjdvsgsgks
Copy link
Author

@yujincheng08 thanks for your suggestion. i didn't know that before and i will try it later. but this feature is still useful for someone who want to stay at edxposed by some reason :D

and my english is much worse than both of yours(you are good enough, please stop fighting). so if there is any incorrect in my replies or code, feel free to point it😋

@XspeedPL
Copy link

XspeedPL commented Feb 7, 2021

I'd say make it a feature that is always enabled.
I don't see why anyone would want this disabled.

Then why not use LSPosed

Being an lsp is hard :(

@MlgmXyysd
Copy link
Member

Sorry, I'm very busy now, the PRs processing of non translation will slow down.

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

Successfully merging this pull request may close these issues.

None yet

4 participants