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

Fix route ChangePasswordConfirmation #10887

Merged

Conversation

DotCat1985
Copy link
Contributor

Mapped the action OrchardCore.Users.Controllers.ChangePasswordConfirmation() to /ChangePasswordConfirmation, because that action was mapped to /OrchardCore.Users/ChangePasswordConfirmation.

@dnfadmin
Copy link

dnfadmin commented Dec 15, 2021

CLA assistant check
All CLA requirements met.

@sebastienros
Copy link
Member

Can you please also update this part of the documentation?

https://github.com/OrchardCMS/OrchardCore/blob/main/src/docs/reference/modules/Users/README.md

Thanks

@DotCat1985
Copy link
Contributor Author

Can you please also update this part of the documentation?

https://github.com/OrchardCMS/OrchardCore/blob/main/src/docs/reference/modules/Users/README.md

Thanks

Done. You can see the documentation update clicking on the link below.

8eb7979

I also commited the renaming of the property ChangePasswordConfirmation to ChangePasswordConfirmationPath, because it represents an action path like the property LogoffPath.

06bb200

@deanmarcussen
Copy link
Member

Rerunning the ci. Changes look good thanks

@agriffard agriffard merged commit 0792276 into OrchardCMS:main Jan 19, 2022
@sebastienros
Copy link
Member

Can you please also update this part of the documentation?

:/

@agriffard
Copy link
Member

https://docs.orchardcore.net/en/latest/docs/reference/modules/Users/#custom-paths

  "OrchardCore": {
    "OrchardCore_Users": {
      "LoginPath": "Login",
      "LogoffPath": "Users/LogOff",
      "ChangePasswordUrl": "ChangePassword",
      "ChangePasswordConfirmationUrl": "ChangePasswordConfirmation",
      "ExternalLoginsUrl": "ExternalLogins"
    }
  }

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.

5 participants