Skip to content
This repository was archived by the owner on Dec 13, 2018. It is now read-only.

Remove givenName and surname restrictions to registration data model#512

Open
rorheden wants to merge 2 commits intomasterfrom
feature-registration-account-model
Open

Remove givenName and surname restrictions to registration data model#512
rorheden wants to merge 2 commits intomasterfrom
feature-registration-account-model

Conversation

@rorheden
Copy link
Copy Markdown
Contributor

@rorheden rorheden commented Aug 28, 2016

Removes givenName and surname restrictions on the registration data model.

Important

This depends on the PR stormpath/stormpath-node-config#61 before it can be merged.

How to verify

Verify that once the application is started the web.register.form.fields configuration (https://github.com/stormpath/express-stormpath/blob/master/lib/config.yml#L65) reflects the account schema of your default directory.

Fixes stormpath/stormpath-sdk-node#526

@rorheden rorheden force-pushed the feature-registration-account-model branch from 174e449 to ae6c1a3 Compare August 29, 2016 11:50
@robertjd robertjd force-pushed the feature-registration-account-model branch from ae6c1a3 to 77898aa Compare September 27, 2016 01:50
@robertjd
Copy link
Copy Markdown
Member

@typerandom FYI that I've rebased this one against master. This one will need a lot of work on the registration controller tests. I pushed one small fix that fixed some of the failing tests. I'm testing while linked to stormpath/stormpath-node-config#61

@rorheden rorheden force-pushed the feature-registration-account-model branch from 77898aa to ae6c1a3 Compare October 8, 2016 11:51
@robertjd robertjd added open and removed in progress labels Oct 17, 2016
@rorheden rorheden added in progress and removed open labels Oct 31, 2016
@rorheden rorheden self-assigned this Oct 31, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants