Commit 8924ba9
authored
fix: pass masterConsentsOrigin as URL param and merge duplicate integrations key (#193)
## Summary
- **termly.js**: `masterConsentsOrigin` was set as a `data-` attribute
instead of a URL query parameter, so Termly never read it and consent
sync between www.kosli.com and docs.kosli.com did not work. Now passed
as `&masterConsentsOrigin=https://www.kosli.com` on the script src URL
per [Termly
docs](https://support.termly.io/hc/en-us/articles/30710516322321).
- **docs.json**: Two duplicate `integrations` keys — the second
(mixpanel) silently overwrote the first (cookies), dropping the cookie
consent gating for Mintlify telemetry. Merged into a single key.
## Test plan
- [ ] Deploy to docs.kosli.com
- [ ] Accept cookies on www.kosli.com
- [ ] Open docs.kosli.com in a new tab — banner should NOT appear
(consent synced)
- [ ] Open docs.kosli.com in incognito — banner SHOULD appear1 parent 85e8b99 commit 8924ba9
2 files changed
Lines changed: 5 additions & 8 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
36 | 36 | | |
37 | 37 | | |
38 | 38 | | |
39 | | - | |
40 | | - | |
41 | | - | |
42 | | - | |
43 | | - | |
44 | | - | |
45 | 39 | | |
46 | 40 | | |
47 | 41 | | |
| |||
79 | 73 | | |
80 | 74 | | |
81 | 75 | | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
82 | 80 | | |
83 | 81 | | |
84 | 82 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
32 | 32 | | |
33 | 33 | | |
34 | 34 | | |
35 | | - | |
36 | | - | |
| 35 | + | |
37 | 36 | | |
38 | 37 | | |
39 | 38 | | |
| |||
0 commit comments