Skip to content

IBX-11100: Hybrid tracking described in Dev-doc#3250

Open
julitafalcondusza wants to merge 6 commits into
5.0from
IBX-11100
Open

IBX-11100: Hybrid tracking described in Dev-doc#3250
julitafalcondusza wants to merge 6 commits into
5.0from
IBX-11100

Conversation

@julitafalcondusza

Copy link
Copy Markdown
Contributor
Question Answer
JIRA Ticket (https://github.com/ibexa/connector-raptor/pull/59)
Versions
Edition

Hybrid tracking described in Dev-doc

Checklist

  • Text renders correctly
  • Text has been checked with vale
  • Description metadata is up to date
  • Redirects cover removed/moved pages
  • Code samples are working
  • PHP code samples have been fixed with PHP CS fixer
  • Added link to this PR in relevant JIRA ticket or code PR

@github-actions

Copy link
Copy Markdown

Preview of modified files

Preview of modified Markdown:

@julitafalcondusza julitafalcondusza marked this pull request as ready for review June 25, 2026 10:38
Comment thread docs/recommendations/raptor_integration/hybrid_tracking.md

@mnocon mnocon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hybrid tracking must be mentioned in other places, it's not enough to describe it just in a separate article - right now it feels disconnected from other parts of the doc.

Examples:

tracking_type: hybrid must be mentioned, in the YAML and in the list

hybrid_tracking_proxy_path should be mentioned in the global configuration reference?

The function output must be expanded to mention the hybrid type

Comment thread docs/recommendations/raptor_integration/hybrid_tracking.md
Comment thread docs/recommendations/raptor_integration/hybrid_tracking.md

When hybrid tracking is enabled, `TrackingScriptExtension` renders the proxy bootstrap template and loads the `raptor-proxy.js` shim, which replaces the `window.raptor.push` and drains the initial event queue.
The shim sends events to the configured proxy endpoint, by default, `/raptor/track`.
Each event is sent in a separate request using a batch-compatible format.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm not sure what it means. What's a batch-compatible format and why is it important to mention?

Comment thread docs/recommendations/raptor_integration/hybrid_tracking.md
Comment thread docs/recommendations/raptor_integration/hybrid_tracking.md
Comment thread docs/recommendations/raptor_integration/hybrid_tracking.md
Comment thread docs/recommendations/raptor_integration/hybrid_tracking.md
### Template usage

By default, the tracking script waits for user consent before sending tracking events.
Consent can be provided either by rendering the script with `ibexa_tracking_script` (`has_consented` = `true`) or by triggering the `enableTracking` JavaScript event.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I feel this part is repeated in https://ez-systems-developer-documentation--3250.com.readthedocs.build/en/3250/templating/twig_function_reference/recommendations_twig_functions/#ibexa_tracking_script-function:~:text=needed%2E-,If,consent,-%2E , but here it's less detailed, doesn't mention enableTracking and doesn't show an example.

Can we just say something like:

By default, the tracking script waits for user consent before sending tracking events. For more info, see https://ez-systems-developer-documentation--3250.com.readthedocs.build/en/3250/templating/twig_function_reference/recommendations_twig_functions/#ibexa_tracking_script-function

And maybe in recommendations_twig_functions add a new subsection called "Handle tracking consent" for easier linking between sections?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Agreed — no point duplicating it here, especially since this version is less complete (no enableTracking, no example). Shortening it to a one-line note plus a link to ibexa_tracking_script makes sense, and adding a "Handle tracking consent" subsection there for easy linking is a good idea.

Comment thread docs/recommendations/raptor_integration/hybrid_tracking.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants