Skip to content

Adds ability to prevent login with unverified emails #2165

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

Closed
wants to merge 2 commits into from
Closed

Adds ability to prevent login with unverified emails #2165

wants to merge 2 commits into from

Conversation

cherukumilli
Copy link
Contributor

Updated README file to show how to use the new config parameter (allowLoginForUnverifiedEmail) that was introduced in PR #2163

var server = ParseServer({
  ...otherOptions,
  // Enable email verification
  verifyUserEmails: true,

  // allows user to login only after email verification
  allowLoginForUnverifiedEmail: false, // defaults to true

  // The public URL of your app.
  // This will appear in the link that is used to verify email addresses and reset passwords.
  // Set the mount path as it is in serverURL
  publicServerURL: 'https://example.com/parse',
...

@flovilmart
Copy link
Contributor

Can you please do both PR as a single? That will help keeping track of everything.

@flovilmart flovilmart closed this Jun 28, 2016
@flovilmart flovilmart reopened this Jun 29, 2016
@flovilmart flovilmart changed the title Updates README file for pr 2163 Adds ability to prevent login with unverified emails Jun 29, 2016
@flovilmart
Copy link
Contributor

Needs a Rebase and unit tests that cover that behavior before we can merge! Otherwise that look good.

@cherukumilli
Copy link
Contributor Author

Closing this PR for now. I will open another one as soon as I get some time to squash the commits and add test cases.

@cherukumilli cherukumilli deleted the README-updates-for-pr-2163 branch June 29, 2016 19:54
@cherukumilli cherukumilli restored the README-updates-for-pr-2163 branch June 29, 2016 19:54
@flovilmart
Copy link
Contributor

Thanks ;) tests are important in order to maintain the feature over time.

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.

3 participants