CIF-1084 - Implement the password reset page

[herzog31]

Description

• Adds a new reset-password page at /content/venia/{language-master|us}/en/reset-password.html.
• Adds a proxy component for new resetpassword component.
• Adds new ResetPassword React component to the React app.

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• I have signed the Adobe Open Source CLA.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes and the overall coverage did not decrease.
• All unit tests pass on CircleCi.
• I ran all tests locally and they pass.

[cjelger]

A generic comment and sorry if I haven't followed precisely: is this "reset password" a "forgot password" functionality that is used when a user forgot her/his password? If yes, then we should probably not have the page under "my-account" because the user cannot login, so I think it might be misleading to have this under "my-account".

[mhaack]

+1 to @cjelger 's comment plus additionally we should make sure if you are navigation to my-account pages/sub-pages without being logged in we should redirect to the homepage and if possible open the login form. Ideally, we should also have a dedicated login page where we would redirect to. But all this is a separate ticket.

[herzog31]

@cjelger @mhaack Magento Luma has it under the account tree. Any suggestions where the page should be?

[cjelger]

Maybe below language-masters/en ? Also make sure the page is hidden in the navigation like the search page (with hideInNav true). I agree with Markus about the redirect.

[herzog31]

@mhaack @cjelger I moved the reset password page to /content/venia/{language-master|us}/en/reset-password.html.

[codecov]

Codecov Report

❗ No coverage uploaded for pull request base (dev@ad58471). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             dev      #60   +/-   ##
======================================
  Coverage       ?   79.16%           
  Complexity     ?       13           
======================================
  Files          ?        1           
  Lines          ?       24           
  Branches       ?        4           
======================================
  Hits           ?       19           
  Misses         ?        3           
  Partials       ?        2           

Flag	Coverage Δ	Complexity Δ
#unittests	79.16% <ø> (?)	13.00 <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ	Complexity Δ
...enia/core/models/commerce/MyProductTeaserImpl.java	79.16% <0.00%> (ø)	13.00% <0.00%> (?%)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ad58471...287c52e. Read the comment docs.