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

BNPL PMME messaging in cart doesn't appear immediately below the Total when Gift Card extension is enabled #8661

Closed
pierorocca opened this issue Apr 17, 2024 · 4 comments · Fixed by #8671
Assignees
Labels
focus: checkout payments priority: high The issue/PR is high priority—it affects lots of customers substantially, but not critically. type: bug The issue is a confirmed bug.

Comments

@pierorocca
Copy link
Contributor

Describe the bug

BNPL PMME messaging in cart doesn't appear immediately below the Total when Gift Card extensions is enabled. I expect this is true for other extensions like Points and Rewards which add a similar field.

image

To Reproduce

  1. Download and enable Gift Card extension
  2. Enable desired BNPL payment methods
  3. Shop and add product to cart
  4. Observe in cart that the PMME messaging is below the gift card code field rather than below the Total

Expected behavior

PMME messaging is always the first element located below the Total

@pierorocca pierorocca added type: bug The issue is a confirmed bug. priority: high The issue/PR is high priority—it affects lots of customers substantially, but not critically. focus: checkout payments labels Apr 17, 2024
@pierorocca pierorocca changed the title BNPL PMME messaging in cart doesn't appear immediately below the Total when Gift Card extensions is enabled BNPL PMME messaging in cart doesn't appear immediately below the Total when Gift Card extension is enabled Apr 17, 2024
@brettshumaker brettshumaker self-assigned this Apr 18, 2024
@brettshumaker
Copy link
Contributor

The GC plugin is inserting their html at an earlier priority (9) than we are (10). I can insert ours earlier (5), but it's still best practice to leave some room for other plugins to hook in where they need to since we couldn't possibly imagine all scenarios where someone would want to add something there.

Screenshot 2024-04-18 at 3 53 36 PM

@pierorocca
Copy link
Contributor Author

Thanks Brett. The two I can think of is gift cards and points and rewards.

Related to BNPL I see Affirm, Afterpay etc. also populating that space.

Is there any governance or standard to determine priority to use?

@brettshumaker
Copy link
Contributor

Is there any governance or standard to determine priority to use?

Not really. The default priority is 10 so it's generally assumed that if it doesn't really matter where your item is placed, you just use the default and adjust from there if you want it earlier/later. The common courtesy is to leave a little space between your position and default or other known uses, otherwise you'll end up relying on plugin load order or have to manually move other plugin's hooks.

@pierorocca
Copy link
Contributor Author

Gotcha. Thanks for the insight.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
focus: checkout payments priority: high The issue/PR is high priority—it affects lots of customers substantially, but not critically. type: bug The issue is a confirmed bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants