Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

namdinh-92
Copy link
Contributor

1. Why is this change necessary?

  • Only showing Paypal error messages in checkout page, not showing in category or other pages

2. What does this change do, exactly?

  • Indicated the suitable flash messages in storefront/resources/views/storefront/base.html.twig and storefront/resources/views/storefront/page/checkout/_page.html.twig to make sure that the paypal errors only showing in checkout page, not showing on other pages.

3. Describe each step to reproduce the issue or behaviour.

  • Make some steps on Storefront during Paypal checkout modal opened and in progress then close it or do something to interrupt that progress

4. Please link to the relevant issues (if any).

5. Checklist

  • I have written tests and verified that they fail without my change
  • I have created a changelog file with all necessary information about my changes
  • I have written or adjusted the documentation according to my changes
  • This change has comments for package types, values, functions, and non-obvious lines of code
  • I have read the contribution requirements and fulfilled them

@namdinh-92 namdinh-92 self-assigned this Oct 15, 2025
@namdinh-92 namdinh-92 added Platform(Default) domain/checkout Responsible for fulfilling the buying transaction inside the store and everything related to it labels Oct 15, 2025
@namdinh-92 namdinh-92 marked this pull request as draft October 15, 2025 15:31
Copy link

Warnings
⚠️ The Pull Request doesn't contain any changelog file

Copy link

codecov bot commented Oct 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 55.72%. Comparing base (aaa5bb7) to head (f0a85d5).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##            trunk   #13030      +/-   ##
==========================================
- Coverage   55.73%   55.72%   -0.01%     
==========================================
  Files        4908     4908              
  Lines      167152   167147       -5     
  Branches     9050     9050              
==========================================
- Hits        93154    93149       -5     
- Misses      71313    71314       +1     
+ Partials     2685     2684       -1     
Flag Coverage Δ
jest-admin 61.14% <ø> (-0.01%) ⬇️
jest-storefront 70.01% <ø> (ø)
phpunit-migration 49.31% <ø> (ø)
phpunit-unit 49.40% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@namdinh-92 namdinh-92 force-pushed the fix/12701-Paypal-errors-showing-on-category-page branch from d4ffda5 to f0a85d5 Compare October 15, 2025 16:16
@namdinh-92 namdinh-92 marked this pull request as ready for review October 16, 2025 06:16
@namdinh-92 namdinh-92 requested review from cyl3x and untilu29 October 16, 2025 06:16
@untilu29 untilu29 requested review from a team and removed request for cyl3x and untilu29 October 16, 2025 06:37
@namdinh-92 namdinh-92 marked this pull request as draft October 16, 2025 06:39
@cyl3x
Copy link
Contributor

cyl3x commented Oct 16, 2025

@namdinh-92 everything regarding the PayPal plugin has to live inside the PayPal Plugin - we don't fix issues with plugin specific code in platform itself

@namdinh-92
Copy link
Contributor Author

@namdinh-92 everything regarding the PayPal plugin has to live inside the PayPal Plugin - we don't fix issues with plugin specific code in platform itself

Yes right, I rethink it for a while and arise another fix, revert this PR to be Draft for now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain/checkout Responsible for fulfilling the buying transaction inside the store and everything related to it milestone/6.7.4.0 Platform(Default)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Paypal error showing on category page?

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.