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

Refactor Custom Fields flow #290

Merged
merged 4 commits into from
Jun 14, 2016
Merged

Refactor Custom Fields flow #290

merged 4 commits into from
Jun 14, 2016

Conversation

lbalmaceda
Copy link
Contributor

  • If the custom fields are less or equal than 2, they are shown in the "first" sign up screen. If they are more, the "second" screen will show all of them.
  • Consider "error" and "code" keys when parsing login errors.
  • Username Style properly handled.

@lbalmaceda lbalmaceda added this to the 2.0.0-beta.3 milestone Jun 8, 2016
@lbalmaceda lbalmaceda changed the title Refactor custom fields screens Refactor Custom Fields flow Jun 8, 2016
@lbalmaceda lbalmaceda force-pushed the refactor-custom-fields-screens branch 2 times, most recently from 1700f9d to fe1c959 Compare June 13, 2016 14:06
return event;
} else if (lockWidget.getConfiguration().hasExtraFields()) {
Copy link
Member

Choose a reason for hiding this comment

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

why else if then returns?

@lbalmaceda lbalmaceda force-pushed the refactor-custom-fields-screens branch from fe1c959 to f8a38ca Compare June 14, 2016 20:39
@lbalmaceda lbalmaceda force-pushed the refactor-custom-fields-screens branch from f8a38ca to 419e601 Compare June 14, 2016 20:45
@hzalaz hzalaz merged commit 67bdfc0 into v2 Jun 14, 2016
@hzalaz hzalaz deleted the refactor-custom-fields-screens branch June 14, 2016 21:26
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