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.

Sign up for GitHub

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

Jump to bottom

#166465376 Fix url sent to the user's email address #76

Merged
merged 1 commit into from
Jun 6, 2019
Merged

#166465376 Fix url sent to the user's email address #76

merged 1 commit into from
Jun 6, 2019

Conversation

IssaIan
Copy link
Contributor

@IssaIan IssaIan commented Jun 5, 2019 •
edited
Loading

What does this PR do?

  • Fixes the url for password reset sent to the user's email

Relevant pivotal tracker story

#166465376

@IssaIan IssaIan force-pushed the bg-passwordreset-166465376 branch from ecde09b to e7c55c9 Compare June 5, 2019 10:06
@IssaIan IssaIan force-pushed the bg-passwordreset-166465376 branch from e7c55c9 to daf9990 Compare June 5, 2019 10:12
@matthenge matthenge self-requested a review June 6, 2019 08:29
matthenge
matthenge approved these changes Jun 6, 2019
View reviewed changes
Copy link
Contributor

@matthenge matthenge left a comment

Choose a reason for hiding this comment

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

LGTM

@IssaIan IssaIan added the finished All reviews have been implemented label Jun 6, 2019
@IssaIan IssaIan requested a review from koechkevin June 6, 2019 10:52
@koechkevin koechkevin merged commit f731aa9 into develop Jun 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matthenge matthenge matthenge approved these changes

@Ogutu-Brian Ogutu-Brian Ogutu-Brian approved these changes

@koechkevin koechkevin Awaiting requested review from koechkevin

Assignees
No one assigned
Labels
finished All reviews have been implemented
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@IssaIan @matthenge @Ogutu-Brian @koechkevin