chore: translate documentation (2026-03-01)#119
Conversation
|
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 (61)
📝 WalkthroughWalkthroughThis pull request updates internationalized documentation across 10 languages (Arabic, German, Spanish, French, Hindi, Italian, Japanese, Korean, Portuguese-Brazilian, and Simplified Chinese) for WooCommerce POS payment gateways. It adds comprehensive Vipps MobilePay gateway documentation, standardizes terminology and localization across existing gateway guides, updates index pages to reference the new gateway, and adds GitHub Actions workflow permissions for content modification. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Mode: changed Languages: ["es","fr","de","ja","pt-BR","ko","it","ar","hi-IN","zh-CN"] Also fix workflow permissions (contents: write) to prevent push failures.
These were incorrectly removed by orphan cleanup because the English source hadn't been merged to main yet at the time.
fde8b37 to
5369025
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: fde8b37434
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| <RequirementItem label="Pasarelas de Pago Web">Al menos un método de pago en línea configurado en WooCommerce</RequirementItem> | ||
| <RequirementItem label="Certificado SSL">HTTPS requerido para procesamiento seguro de pagos</RequirementItem> | ||
| <RequirementItem label="Navegador Moderno">El dispositivo del cliente debe soportar estándares web modernos</RequirementItem> | ||
| <RequirementItem label="WooCommerce POS">Se requiere versión Pro para el pago en POS</RequirementItem> |
There was a problem hiding this comment.
Keep WooCommerce POS requirement aligned with source docs
This change rewrites the requirement to “Pro version required,” but the canonical English page for the same document still states Free or Pro version installed and activated (versioned_docs/version-1.x/payment/custom-gateways/web-checkout.mdx, line 161). The same semantic regression appears across the translated custom-gateway pages in this commit, so non-English users are now given incorrect licensing guidance and may incorrectly assume these gateways are unavailable on Free.
Useful? React with 👍 / 👎.
Summary
permissions: contents: writeto translate workflow to prevent future push failuresRoot cause
The translate workflow's
GITHUB_TOKENlacked write permissions, causinggit pushto fail with a 403. Fixed by adding top-levelpermissions: contents: write.Summary by CodeRabbit
Release Notes
Documentation
Chores