wp-calypso
wp-calypso copied to clipboard
A4A: update unpaid invoice banner
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:
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 setstatus => '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)?