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

Licensing Portal: Improve no primary payment method error clarity #65291

Merged

Conversation

atanas-dev
Copy link
Member

@atanas-dev atanas-dev commented Jul 6, 2022

Proposed Changes

  • Licensing Portal: Improve no primary payment method error clarity

Testing Instructions

  • Switch your account to an agency and your testing key billing type to "Stripe".
  • Make sure you don't have a payment method.
  • Click on the "Assign" on any unassigned license and confirm the wording makes sense.
  • Apply this diff so this edge case is easy to trigger:
diff --git a/client/jetpack-cloud/sections/partner-portal/issue-license-form/index.tsx b/client/jetpack-cloud/sections/partner-portal/issue-license-form/index.tsx
index 33c608f46c..b4c1abfc34 100644
--- a/client/jetpack-cloud/sections/partner-portal/issue-license-form/index.tsx
+++ b/client/jetpack-cloud/sections/partner-portal/issue-license-form/index.tsx
@@ -128,7 +128,7 @@ export default function IssueLicenseForm( {
 
 			switch ( error.code ) {
 				case 'missing_valid_payment_method':
-					if ( paymentMethodRequired ) {
+					if ( false && paymentMethodRequired ) {
 						page.redirect(
 							addQueryArgs(
 								{

@atanas-dev atanas-dev added Jetpack Cloud Anything related to the Jetpack Cloud (cloud.jetpack.com) Jetpack Licensing Portal labels Jul 6, 2022
@atanas-dev atanas-dev self-assigned this Jul 6, 2022
@matticbot
Copy link
Contributor

This PR does not affect the size of JS and CSS bundles shipped to the user's browser.

Generated by performance advisor bot at iscalypsofastyet.com.

Copy link
Contributor

@heydemoura heydemoura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM :shipit:

@atanas-dev atanas-dev merged commit e14815c into trunk Jul 7, 2022
@atanas-dev atanas-dev deleted the update/partner-portal-payment-method-error-wording branch July 7, 2022 12:20
@a8ci18n
Copy link

a8ci18n commented Jul 7, 2022

This Pull Request is now available for translation here: https://translate.wordpress.com/deliverables/7423264

Thank you @atanas-dev for including a screenshot in the description! This is really helpful for our translators.

@a8ci18n
Copy link

a8ci18n commented Jul 14, 2022

Translation for this Pull Request has now been finished.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Jetpack Cloud Anything related to the Jetpack Cloud (cloud.jetpack.com) Jetpack Licensing Portal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants