Skip to content

Added Android Widget #2614

Open
adjuster2004 wants to merge 7 commits into
amnezia-vpn:devfrom
adjuster2004:widget
Open

Added Android Widget #2614
adjuster2004 wants to merge 7 commits into
amnezia-vpn:devfrom
adjuster2004:widget

Conversation

@adjuster2004
Copy link
Copy Markdown

Added Android Widget

adjuster2004 and others added 7 commits May 2, 2026 21:50
Added a list of applications that should be included in the VPN
Added a list of applications that should be included in the VPN
Added a list of applications that should be included in the VPN
Added a list of applications that should be included in the VPN
Added a list of applications that should be included in the VPN
@ygurov
Copy link
Copy Markdown
Collaborator

ygurov commented May 18, 2026

Hi @adjuster2004
Thanks for this PR. But, unfortunately, I doubt it could be even considered to be merged in it's current state.

I would recommend to do the following:

  1. Avoid unnecessary changes (i.e. refomating CMakeLists.txt, touching jsons and adding redundant comments)
  2. Keep your changes isolated to the PR topic. If any changes are really related to each other, add some comments about that in PR description
  3. Get rid of local development/neuroslope comments in the code. Nobody except you would ever understand all of those strings in the code
  4. Keep all the code in English (except translations)

Neat and tidy PRs could be reviewed more easily, the more chances they would be merged

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