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

[Shop][Checkout] Sigining in during checkout addressing step #5255

Conversation

Arminek
Copy link
Contributor

@Arminek Arminek commented Jun 14, 2016

Q A
Bug fix? no
New feature? yes
BC breaks? no
License MIT

Create authentication handlers for Ajax signing in.

@Arminek Arminek force-pushed the new-shop-checkout-addressing-as-existing-customer branch from f4cd5ab to 6998339 Compare June 14, 2016 23:13
@pjedrzejewski pjedrzejewski added New Feature Shop ShopBundle related issues and PRs. labels Jun 14, 2016
@Arminek Arminek force-pushed the new-shop-checkout-addressing-as-existing-customer branch from 6998339 to 9fa38f1 Compare June 14, 2016 23:37
@Arminek Arminek changed the title [Shop] Sigining in during checkout addressing step [Shop][Checkout] Sigining in during checkout addressing step Jun 14, 2016
I want to be able to sign in and fill addressing details

Background:
Given the store operates on a single channel in "France"
Copy link
Member

Choose a reason for hiding this comment

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

Wrong indents here and below.

@Arminek Arminek force-pushed the new-shop-checkout-addressing-as-existing-customer branch 3 times, most recently from cd9c63c to 91ba4cf Compare June 15, 2016 12:39
{{ form_row(form.customer.email) }}
<div class="one field" id="apiLogin">
{{ form_row(form.customer.email) }}
<div class="ui action input" id="apiToggleable">
Copy link
Member

Choose a reason for hiding this comment

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

This id is way too general.

@Arminek Arminek force-pushed the new-shop-checkout-addressing-as-existing-customer branch from 91ba4cf to 9d9ee57 Compare June 16, 2016 07:22
@Arminek Arminek force-pushed the new-shop-checkout-addressing-as-existing-customer branch from 9d9ee57 to 8960ba0 Compare June 16, 2016 10:14
@pjedrzejewski pjedrzejewski merged commit 4dd4f1c into Sylius:master Jun 16, 2016
@pjedrzejewski
Copy link
Member

Thanks Arek, great work! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Shop ShopBundle related issues and PRs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants