Skip to content
This repository has been archived by the owner on Jun 3, 2021. It is now read-only.

Feature auth support #20

Closed
wants to merge 3 commits into from

Conversation

168668660
Copy link
Contributor

module和component鉴权接口android和ios都已添加

@sospartan
Copy link
Contributor

@zshshr @cxfeng1 pls review this PR.

@168668660

  1. It's 'Validate' not 'auth', change this PR's title.
  2. Do not commit with chinese any longer, it's not meet our contribute requirement. Thanks. 😄

@@ -294,6 +294,11 @@
private boolean isCommit=false;
private WXGlobalEventReceiver mGlobalEventReceiver=null;
private boolean trackComponent;
private boolean needValidate = false;
Copy link
Contributor

@zshshr zshshr Mar 6, 2017

Choose a reason for hiding this comment

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

“ needValidate” member variable should start with ‘m’ ,like “mNeedValidate”

@sospartan
Copy link
Contributor

@168668660 pls fix the conflicts.

@cxfeng1 pls review the iOS part.

@168668660 168668660 closed this Mar 7, 2017
@168668660 168668660 deleted the feature-auth-support branch March 7, 2017 09:27
@sospartan
Copy link
Contributor

@168668660 what's going on?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants