wp-calypso icon indicating copy to clipboard operation
wp-calypso copied to clipboard

A4A: update unpaid invoice banner

Open andrii-lysenko opened this issue 4 months ago • 3 comments

Resolves https://github.com/Automattic/automattic-for-agencies-dev/issues/1230

Proposed Changes

Currently, we have Payment notice banner available on Matketplace pages. However, it's generic at the moment. We want to make it more clear for users that they need to pay their bill. We are forcing this banner on all pages and make it non dismissible. We also have different messaging dependant on how many days overdue.

Here are banners:

Screenshot 2024-10-18 at 11 27 18 AM Screenshot 2024-10-18 at 11 27 37 AM Screenshot 2024-10-18 at 11 27 55 AM Screenshot 2024-10-18 at 11 28 12 AM Screenshot 2024-10-18 at 11 28 27 AM

Why are these changes being made?

Testing Instructions

  • You would need to patch this diff (D164082-code) to your sandbox (or use trunk if already merged). Point public-api.wordpress.com you your sandbox IP in your hosts file.
  • The easiest way to test if you don't have overdue invoices:
  • In your sandbox, navigate to wp-content/lib/jetpack-licensing/rest-api/class-invoice-resource.php and set status => 'open'.
  • In the same file change created to some past date eg (strtotime(Today - 3 days), strtotime(Today - 20 days).
  • Check different pages on live link of this pr, including overview, marketplace etc.

Pre-merge Checklist

  • [ ] Has the general commit checklist been followed? (PCYsg-hS-p2)
  • [ ] Have you written new tests for your changes?
  • [ ] Have you tested the feature in Simple (P9HQHe-k8-p2), Atomic (P9HQHe-jW-p2), and self-hosted Jetpack sites (PCYsg-g6b-p2)?
  • [ ] Have you checked for TypeScript, React or other console errors?
  • [ ] Have you used memoizing on expensive computations? More info in Memoizing with create-selector and Using memoizing selectors and Our Approach to Data
  • [ ] Have we added the "[Status] String Freeze" label as soon as any new strings were ready for translation (p4TIVU-5Jq-p2)?
  • [ ] For changes affecting Jetpack: Have we added the "[Status] Needs Privacy Updates" label if this pull request changes what data or activity we track or use (p4TIVU-aUh-p2)?

andrii-lysenko avatar Oct 17 '24 22:10 andrii-lysenko