Repository navigation
Record the company a BIQ visitor opens - #73
Conversation
Existing biq events stay. kld_company is sent as well, for the bank and for the examples page.
|
@codex review |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
There was a problem hiding this comment.
🟡 Changes recommended
The standalone pages execute an unpinned dependency from a mutable branch without subresource integrity.
2 open findings
What changed in this PR
Adds shared analytics tracking for BIQ application views and company selection.
Changes:
- Emits
app_viewandkld_companyevents. - Tracks initial and picker-selected companies.
- Loads the app-kit analytics helper on standalone pages.
| File | Description |
|---|---|
js/biq.js |
Adds bank and company-selection analytics. |
js/biq-examples.js |
Adds examples-page company analytics. |
index.html |
Loads the shared analytics helper. |
examples.html |
Loads the shared analytics helper. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
jsDelivr at main can change without a BIQ review. Both pages now load that commit with subresource integrity.
|
@codex review |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
@codex review |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Record the company a visitor opens in BIQ.
biq_company,biq_level,biq_search,biq_examples_view, andbiq_examples_openstay as they are.kld_companyis sent as well: the company in the URL on load, a pick in the company control, and the company on the examples page.app_viewfires once per load. The helper iskldTrackfrom kindel/app-kit.The standalone pages pin the helper to app-kit commit
1e80086c9a4cfd40493e0ca44629fb9b96cf81f1, with subresource integrity andcrossorigin="anonymous". The kindel.com page shell is in kindelwww, so the later pin onkindelwwwmaster adds<script src="{{ "js/app-kit/analytics.js" | relURL }}" defer></script>beforejs/biq.jsand beforejs/biq-examples.js, and mountsanalytics.js.No PII. No new cookies.
localStorageholds company ids only, underkld-company-set.kindelwwwcontent/privacy.mdstill matches this.Merge kindel/app-kit's analytics pull request first, then this one.