Conversation
… consistency by changing "à dataLayer" to "au dataLayer" for better localization accuracy.
Improve french translation
fix: prevent null pointer error in get_plugin() for PHP 8.0+
feat: improve gtm events translation
Dependabot couldn't find the original pull request head commit, df76dfd.
….1.3 Bump pbkdf2 from 3.1.2 to 3.1.3
chore: fix package.json / webpack-dev-server vulnerabilities
feat: ✨ Add UTM parameters to signup link
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
1 issue found across 8 files
Prompt for AI agents (all 1 issues)
Understand the root cause of the following 1 issues and fix them.
<file name="package.json">
<violation number="1" location="package.json:63">
This resolution forces webpack-dev-server v5, but laravel-mix 6 expects v4. Forcing the higher major version risks breaking Mix’s dev server workflow; please align the resolution with the supported v4 series.</violation>
</file>
React with 👍 or 👎 to teach cubic. Mention @cubic-dev-ai to give feedback, ask questions, or re-run the review.
feat: maj readme according ticket 76771
There was a problem hiding this comment.
Reviewed changes from recent commits (found 1 issue).
1 issue found across 1 file
Prompt for AI agents (all 1 issues)
Understand the root cause of the following 1 issues and fix them.
<file name="readme.txt">
<violation number="1" location="readme.txt:38">
Add a space before the bolded "Optimization" text so the Markdown renders correctly and remains readable.</violation>
</file>
React with 👍 or 👎 to teach cubic. Mention @cubic-dev-ai to give feedback, ask questions, or re-run the review.
- Fix initialization bug (axeptioSettings → axeptioWpConsentCategories) - Add 5 WP Consent API categories as virtual vendors - Update consent directly without Google Consent Mode dependency - Refactor categories data to single source of truth - Add WordPress favicon for consent categories
feat: authorize Axeptio on WordPress version 6.9
…to-scroll into the drawer)
feat: PLG-31 - custom context banner
feat: PLG-130 - add WP Consent API categories integration
Purely cosmetic. Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
|
Hey @Loubal70, A few things to move this forward: 1. Cubic thread to resolve Could you take a look and resolve it when you get a chance? Once that's cleared, we should be good to merge 2. Next steps after merge — I need to understand the workflow a. After merging into b. What is the environment at https://wordpress-843309-3452223.cloudwaysapps.com/wp-admin — is that staging/preprod? And who is responsible for deploying there? I want to push this release into production. Thanks! |
|
Hi, Thanks for the detailed message. I'm currently on vacation, so I won't be able to look into this right away — I'll try to take a quick look tomorrow morning though. Regarding the PR, I've set it to draft for now. We're waiting on @ogorzalka , who needs to set up automatic deployment and document the release process. At the moment, he's the only person able to push a release to production. As for the workflow, I'd leave it up to Olivier to define the most appropriate approach. Personally, I think we could use About the environment at the Cloudways URL you mentioned — it's currently managed by Amphibee, and it doesn't have an automated deployment pipeline either. The plan is to set up automatic deployment when a PR is merged into the preprod branch (which will need to be protected, of course). I'll get back to you soon. Best, |
There was a problem hiding this comment.
4 issues found across 34 files
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="languages/axeptio-wordpress-plugin-es_ES.po">
<violation number="1" location="languages/axeptio-wordpress-plugin-es_ES.po:536">
P3: The new URL string is flagged as fuzzy, which causes the Spanish translation to be ignored at runtime. Remove the fuzzy flag and update the localized URL so the Spanish build doesn’t fall back to the English link.</violation>
</file>
<file name="readme.txt">
<violation number="1" location="readme.txt:21">
P3: Fix the missing space between sentences so the readme text renders correctly.</violation>
</file>
<file name="assets/js/backend/components/pluginList.js">
<violation number="1" location="assets/js/backend/components/pluginList.js:251">
P2: For non-default languages, getLocalizedMeta falls back to the raw meta string when parsing fails or the parsed value isn’t an object. This shows the default text instead of '', preventing PO/MO translation fallback for missing translations. Return '' when lang !== 'default' in these fallbacks.
(Based on your team's feedback about not falling back to default localized values.) [FEEDBACK_USED]</violation>
</file>
<file name="templates/admin/onboarding/account.php">
<violation number="1" location="templates/admin/onboarding/account.php:13">
P2: Add rel="noopener noreferrer" when using target="_blank" to prevent tabnabbing.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
…is empty in axeptio api
fix: filter vendors without consent requirements from recommended settings
There was a problem hiding this comment.
Pull request overview
Release 2.6.2 for the Axeptio WordPress plugin, focusing on WP Consent API integration, multilingual/customizable shortcode placeholders, and various UX/documentation/dependency updates.
Changes:
- Add WP Consent API category “virtual vendors” and synchronize consent via
wp_set_consent. - Add multilingual + customizable shortcode placeholder fields (title/description/button text) and a “hide decoration” option (incl. DB migration + admin UI + frontend rendering).
- Update admin UI styling/behavior (amber theme tweaks, smoother edit drawer) and bump release metadata/docs/dependencies.
Reviewed changes
Copilot reviewed 30 out of 35 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| templates/frontend/shortcode-placeholder.php | Adds per-plugin button text and optional hiding of placeholder decoration. |
| templates/admin/plugins/fields/repeater.php | Updates focus/brand color classes to amber. |
| templates/admin/plugins/fields/modal-delete.php | Updates modal focus ring styling to amber. |
| templates/admin/plugins/fields/edit.php | Adds multilingual placeholder editor + hide-decoration toggle + improved drawer transitions/scroll behavior. |
| templates/admin/onboarding/account.php | Updates signup link with UTM parameters. |
| templates/admin/main/tabs.php | Updates select focus styles to amber. |
| templates/admin/common/fields/toggle.php | Improves toggle layout and hides checkbox via sr-only. |
| templates/admin/common/fields/textarea.php | Updates textarea focus styles to amber. |
| templates/admin/common/fields/text.php | Updates input focus styles to amber. |
| templates/admin/common/fields/image-upload.php | Updates button focus ring styling to amber. |
| readme.txt | Updates stable tag/tested-up-to and adds 2.6.2 changelog + refreshed marketing copy. |
| package.json | Bumps webpack-dev-server to v5 and adds resolutions. |
| languages/axeptio-wordpress-plugin.pot | Updates POT metadata and adds new WP Consent API category strings. |
| languages/axeptio-wordpress-plugin-fr_FR.po | Updates French translations and strings (incl. WP Consent API categories). |
| languages/axeptio-wordpress-plugin-fr_FR.mo | Compiled French translations update. |
| languages/axeptio-wordpress-plugin-es_ES.po | Updates Spanish translations and strings (incl. WP Consent API categories). |
| languages/axeptio-wordpress-plugin-es_ES.mo | Compiled Spanish translations update. |
| languages/axeptio-wordpress-plugin-en_US.po | Updates English PO metadata/strings (incl. WP Consent API categories + URL change). |
| languages/axeptio-wordpress-plugin-en_US.mo | Compiled English translations update. |
| languages/axeptio-wordpress-plugin-de_DE.po | Updates German translations and strings (incl. WP Consent API categories). |
| languages/axeptio-wordpress-plugin-de_DE.mo | Compiled German translations update. |
| includes/classes/utils/class-search-callback-file-location.php | Fixes possible null handling when extracting plugin name. |
| includes/classes/models/class-wp-consent-api-settings.php | Adds category metadata + exposes categories + creates “virtual vendor” representations. |
| includes/classes/models/class-recommended-plugin-settings.php | Skips hooks missing consent requirements. |
| includes/classes/models/class-plugins.php | Adds new meta defaults, normalizes types, and adds localized-meta decoding helper. |
| includes/classes/migrations/class-migration-2.6.2.php | Adds DB columns for new placeholder fields. |
| includes/classes/frontend/class-hook-modifier.php | Uses localized placeholder metas and passes new placeholder settings to frontend. |
| includes/classes/frontend/class-axeptio-sdk.php | Injects WP Consent API vendors/categories into localized frontend configuration. |
| axeptio-wordpress-plugin.php | Bumps plugin version constants to 2.6.2. |
| assets/js/frontend/axeptio.js | Syncs Axeptio choices to WP Consent API categories directly via wp_set_consent. |
| assets/js/backend/components/pluginList.js | Adds client-side helpers to read/write localized metas stored as JSON strings. |
| assets/css/frontend/main.css | Adjusts placeholder icon size on large screens. |
| Taskfile.yml | Removes an extra blank line. |
| README.md | Updates version badge and notes WP Consent API compatibility. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Force webpackbar ^7.0.0 via resolutions to fix ProgressPlugin invalid options error (name, color, reporters) introduced by webpackbar 5.x/6.x being incompatible with recent webpack 5.
- Use Axeptio_SDK instead of axeptioSettings to avoid undefined reference due to script execution order - Use window.wp_set_consent to fix no-undef ESLint error - Return empty string for non-default languages in getLocalizedMeta fallback to allow PO/MO translation fallback
- Fix missing spaces in readme.txt formatting - Add rel="noopener noreferrer" to target="_blank" links in onboarding - Remove fuzzy flags on support URL translations (de, en, es, fr)
- Replace DROP COLUMN IF EXISTS with INFORMATION_SCHEMA checks for MySQL 5.7 compatibility in migration 2.6.2 - Prefix template variables with axeptio_ to satisfy PrefixAllGlobals - Add phpcs:disable/enable blocks for migration schema queries - Auto-fix formatting and coding standards violations
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 30 out of 36 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…fields When setLocalizedMeta converts a plain string to JSON for multilingual storage, preserve the existing value under the 'default' key to prevent data loss for pre-existing installations.
Summary by cubic
Release 2.6.2 adds WP Consent API integration with auto-synced consent categories shown as virtual vendors, a multilingual and customizable shortcode placeholder (incl. button text and hideable decoration), and a smoother plugin edit drawer. It also updates the signup link with UTM, bumps “Tested up to” to 6.9, fixes a PHP 8+ null pointer and a WP Consent API init bug, and improves French GTM event translations.
New Features
Dependencies
Written for commit b8ccf91. Summary will update on new commits.