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

Plugin 1.3.4 #2431

Merged
merged 14 commits into from
Feb 7, 2020
Merged

Plugin 1.3.4 #2431

merged 14 commits into from
Feb 7, 2020

Conversation

Mygod
Copy link
Contributor

@Mygod Mygod commented Feb 4, 2020

Main change: support plugin aliases proposed by @CzBiX: shadowsocks/v2ray-plugin-android#18 (comment)
For other changes, see CHANGES.md.

I have not thoroughly tested this. @madeye Can you help? (client app should not require a plugin lib update)

@madeye
Copy link
Contributor

madeye commented Feb 4, 2020

To test this change, I think we need to update the plugin app accordingly?

@Mygod
Copy link
Contributor Author

Mygod commented Feb 4, 2020

Done.

@Mygod
Copy link
Contributor Author

Mygod commented Feb 5, 2020

To test 4a6dc80, we could test against GoQuiet-android.

@Mygod Mygod marked this pull request as ready for review February 5, 2020 04:45
@madeye
Copy link
Contributor

madeye commented Feb 6, 2020

I tested it locally and found one issue.

In the profile fragment, after switching the plugin, the option didn't get updated. Then saving the profile and open the profile again, the plugin option is updated.

Copy link
Contributor

@madeye madeye left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@Mygod
Copy link
Contributor Author

Mygod commented Feb 6, 2020

Cool. Let's merge this for the next release.

@madeye madeye merged commit 1eb0842 into master Feb 7, 2020
@Mygod Mygod deleted the plugin-1.3.4 branch February 17, 2020 01:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants