Skip to content

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.

3 participants