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

saved address (from customer) now populates fields in checkout #551

Merged
merged 1 commit into from
Jun 20, 2016

Conversation

JSin
Copy link
Contributor

@JSin JSin commented Jun 16, 2016

Some assumptions:
Grabbed data from customer instead of person object. i.e. Use saved company address if exists else use saved account address if exists.
Changed the default checkbox option for "Ship to my billing address" to unchecked.

@CLAassistant
Copy link

CLAassistant commented Jun 16, 2016

CLA assistant check
All committers have signed the CLA.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

@anders-jenkins
Copy link

Can one of the admins verify this patch?

@suutari-ai
Copy link
Contributor

test this please

@suutari-ai
Copy link
Contributor

The above comment was for Jenkins btw.

@anders-jenkins
Copy link

Refer to this link for build results (access rights to CI server needed):
http://spartacus.aintra:8080//job/shoop-ng-pull-requests/2042/
Test PASSed.

@suutari-ai
Copy link
Contributor

Nice work @JSin! Thanks!

@suutari-ai suutari-ai merged commit 67fe97c into shuup:master Jun 20, 2016
suutari-ai added a commit that referenced this pull request Jun 20, 2016
* checkout-initial-address:
  Front/Checkout: Simplify initial address population
  saved address (from customer) now populates fields in checkout
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.

None yet

4 participants