-
Notifications
You must be signed in to change notification settings - Fork 16.4k
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
Sip008 initial support #2942
Sip008 initial support #2942
Conversation
The quick test is passed,base example fie at https://github.com/shadowsocks/shadowsocks-android/blob/master/.github/doc-json.md#appendix-sample-exported-json-file 。 👍 Looks good to me, but someone else must approve. |
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.
Other refactoring may be carried out in v5
.
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.
I think it's ready for merging.
- Deserialize the configuration using a constructor to set default values and initialize objects - Re-enable the UI when online config update failed
proposed by Stzx
Please follow the guide below
You will be asked some questions, please read them carefully and answer honestly
Put an
x
into all the boxes [ ] relevant to your pull request (like that [x])Use Preview tab to see how your pull request will actually look like
Searched for similar pull requests
Compiled the code with Visual Studio
Require translation update
Require document update (readme.md, wikipage, etc)
What is the purpose of your pull request?
Description of your pull request and other information
Close #1703
Close #2939
Close #2928
Finally, here we are......
Some glitch is expected...