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

replace /billing link which leads to 404 #111

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

michaelrichey
Copy link

@michaelrichey michaelrichey commented Dec 11, 2024

Two KG pages had the link:
https://www.wanikani.com/account/subscription/billing

Which leads to 404

  • replace 404 link with link to subscription page
  • update copy to reflect how to accurately perform tasks
  • update screenshots

Copy link

netlify bot commented Dec 11, 2024

Deploy preview succeeded. Preview with username: crabigator password: googlebegone

Name Link
🔨 Latest commit 8188f65
🔍 Latest deploy log https://app.netlify.com/sites/wanikani-knowledgebase/deploys/675a0c7bfff5ec0008454f30
😎 Deploy Preview https://deploy-preview-111--wanikani-knowledgebase.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@michaelrichey
Copy link
Author

@rachelcgrant This is ready for review.
In grabbing new screenshots to replace the old billing page images, I noticed that the prompt for updating details changes depending on whether or not you have an active subscription. It's either:

  • manage details
    or
  • update details

So I included both screens and copy for both. However, this may not matter that much. I was trying to be accurate in my description, but let me know if it's confusing instead. It may be enough to just say "click the button to manage your payment or subscription details" and not have a screenshot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant