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

(feat): New Profile Address page #499

Merged
merged 42 commits into from
Feb 14, 2019

Conversation

kieckhafer
Copy link
Member

Resolves #245
Impact: minor
Type: feature

Issue

Create a profile address book page

Solution

Create a profile address book page. This includes adding a new route, /profile/address.

Breaking changes

None

Testing

  • Visit /profile/address as a non-logged in user, and see a 404 / page not found
  • Visit /profile/address as a logged in use, and see a page that matches our design: https://zpl.io/2vYPqk7
    • This page includes 3 new components (AddressBook, InPageMenuItem, ProfileInfo), and various other new components inside these new components.
  • Confirm new "Profile" button in user menu when logged in
  • Confirm you can view, add, update, and delete addresses.

kieckhafer and others added 30 commits October 8, 2018 14:59
…ommerce/reaction-next-starterkit into feat-kieckhafer-newAccountPage
kieckhafer and others added 8 commits February 12, 2019 22:56
Signed-off-by: Erik Kieckhafer <ek@ato.la>
Signed-off-by: Erik Kieckhafer <ek@ato.la>
Signed-off-by: Erik Kieckhafer <ek@ato.la>
Signed-off-by: Erik Kieckhafer <ek@ato.la>
Signed-off-by: Erik Kieckhafer <ek@ato.la>
…tioncommerce/reaction-next-starterkit into feat-kieckhafer-profileAddressPage
Copy link
Member

@mikemurray mikemurray left a comment

Choose a reason for hiding this comment

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

👍

@mikemurray mikemurray merged commit c333e9d into develop Feb 14, 2019
@mikemurray mikemurray deleted the feat-kieckhafer-profileAddressPage branch February 14, 2019 18:31
@jeffcorpuz jeffcorpuz mentioned this pull request Mar 5, 2019
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.

4 participants