Skip to content

Commit

Permalink
fix: changed reset-link-sent wording and renamed renderSuccss() to re…
Browse files Browse the repository at this point in the history
…setLinkMessage()
  • Loading branch information
zg009 committed Mar 15, 2024
1 parent 8f88e39 commit 7fb0d12
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion default-views/auth/reset-link-sent.hbs
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
</div>

<div class="alert alert-success">
<p>A Reset Password link has been sent to your email.</p>
<p>A Reset Password link has been from the associated email account.</p>
</div>
</div>
</body>
Expand Down
6 changes: 3 additions & 3 deletions lib/requests/password-reset-email-request.js
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ class PasswordResetEmailRequest extends AuthRequest {
.then(() => request.validate())
.then(() => request.loadUser())
.then((userAccount) => request.sendResetLink(userAccount))
.then(() => request.renderSuccess())
.then(() => request.resetLinkMessage())
.catch((error) => request.error(error))
}

Expand Down Expand Up @@ -126,7 +126,7 @@ class PasswordResetEmailRequest extends AuthRequest {
if (!exists) {
// For security reason avoid leaking error information
// See: https://github.com/nodeSolidServer/node-solid-server/issues/1770
return this.renderSuccess()
return this.resetLinkMessage()
}

const userData = { username }
Expand Down Expand Up @@ -197,7 +197,7 @@ class PasswordResetEmailRequest extends AuthRequest {
/**
* Displays the 'your reset link has been sent' success message view
*/
renderSuccess () {
resetLinkMessage () {
this.response.render('auth/reset-link-sent')
}
}
Expand Down

0 comments on commit 7fb0d12

Please sign in to comment.