docs: add Vipps MobilePay gateway page#118
Conversation
- Update gateway name references to "WCPOS Vipps MobilePay" - Fix plugin link to point to Checkout with Vipps MobilePay (was linking to the recurring payments plugin) - Remove Sweden from supported markets (uses Swish, not Vipps) - Clarify WCPOS Pro requirement (needed for POS, gateway also works on standard WooCommerce checkout) - Improve "no need to enable" note wording
All custom payment gateways require Pro for POS checkout. Update wording from "Free or Pro version installed and activated" to "Pro version required for POS checkout" across Stripe Terminal, SumUp Terminal, Email Invoice, Web Checkout, and the index page.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review infoConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughStandardizes WooCommerce POS requirement wording to state "Pro version required for POS checkout" across multiple gateway docs, adds a new Vipps MobilePay gateway documentation page, and updates the versioned sidebar to include the new page. Changes
Sequence Diagram(s)(omitted — changes are documentation and sidebar updates without new runtime control flow) Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@versioned_docs/version-1.x/payment/custom-gateways/vipps-mobilepay.mdx`:
- Around line 68-72: The gateway label is inconsistent between steps (one place
uses "WCPOS Vipps MobilePay" and another uses "Vipps MobilePay"); update the
StepCard title and any step text to use a single, consistent label (choose
either "WCPOS Vipps MobilePay" or "Vipps MobilePay")—for example, change the
StepCard title="Enable in WooCommerce POS" and the step list entry that finds
the gateway to reference the exact same label "Vipps MobilePay" to keep wording
consistent across the setup steps.
ℹ️ Review info
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (7)
versioned_docs/version-1.x/payment/custom-gateways/email-invoice.mdxversioned_docs/version-1.x/payment/custom-gateways/index.mdxversioned_docs/version-1.x/payment/custom-gateways/stripe-terminal.mdxversioned_docs/version-1.x/payment/custom-gateways/sumup-terminal.mdxversioned_docs/version-1.x/payment/custom-gateways/vipps-mobilepay.mdxversioned_docs/version-1.x/payment/custom-gateways/web-checkout.mdxversioned_sidebars/version-1.x-sidebars.json
Summary
Test plan
/payment/custom-gateways/vipps-mobilepay/and verify the page renders with all components (feature grid, installation steps, accordion, requirements list, markets table)/payment/custom-gateways/and verify the index page shows the new "Mobile Payment Gateways" section with the Vipps link🤖 Generated with Claude Code
Summary by CodeRabbit