Skip to content

/ commerce

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

Jump to bottom

Checkout complete page improvements #749

Open
wants to merge 1 commit into
base: 8.x-2.x
from
Open

Checkout complete page improvements #749

wants to merge 1 commit into from

Conversation

@gauravjeet
Copy link

@gauravjeet gauravjeet commented Jun 21, 2017

https://www.drupal.org/node/2869452
@gauravjeet
changed order confirmation message
Loading status checks…
8abae28
@mglaman
Copy link
Member

@mglaman mglaman commented Jun 22, 2017

The path needs to use .id, as the order number may not be the ID. Tests pass because by default it is the same.
@bojanz bojanz force-pushed the drupalcommerce:8.x-2.x branch from dc443c7 to e290888 Jul 27, 2017
@bojanz bojanz force-pushed the drupalcommerce:8.x-2.x branch from 5b3779b to 2941d08 Aug 10, 2017
@haringsrob
Copy link
Contributor

@haringsrob haringsrob commented Sep 12, 2017

I think we can combine the texts of this PR with #788?
@bojanz bojanz force-pushed the drupalcommerce:8.x-2.x branch from 53ec70a to 3bc1b88 Sep 12, 2017
@haringsrob haringsrob mentioned this pull request Sep 14, 2017
Open
@bojanz bojanz force-pushed the drupalcommerce:8.x-2.x branch from 55280a3 to 4e9a025 Jun 15, 2018
@bojanz bojanz force-pushed the drupalcommerce:8.x-2.x branch from b1be14f to 59230dd Jun 28, 2018
@bojanz bojanz force-pushed the drupalcommerce:8.x-2.x branch from 807c6e8 to 8898142 Oct 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
@gauravjeet @mglaman @haringsrob
You can’t perform that action at this time.