Skip to content

feat: enhance telemetry handling for sub-editors#2017

Merged
harbournick merged 2 commits intomainfrom
andrii/sd-1890-telemetry-disable-telemetry-for-sub-editors
Feb 13, 2026
Merged

feat: enhance telemetry handling for sub-editors#2017
harbournick merged 2 commits intomainfrom
andrii/sd-1890-telemetry-disable-telemetry-for-sub-editors

Conversation

@andrii-harbour
Copy link
Contributor

  • Added logic to skip telemetry initialization for editors in 'text' mode and header/footer editors.
  • Updated tests to verify that telemetry is correctly skipped for these cases while still allowing it for other modes like 'docx'.
  • Ensured telemetry configuration is consistent across relevant components.

- Added logic to skip telemetry initialization for editors in 'text' mode and header/footer editors.
- Updated tests to verify that telemetry is correctly skipped for these cases while still allowing it for other modes like 'docx'.
- Ensured telemetry configuration is consistent across relevant components.
@andrii-harbour andrii-harbour self-assigned this Feb 13, 2026
@linear
Copy link

linear bot commented Feb 13, 2026

@andrii-harbour andrii-harbour marked this pull request as ready for review February 13, 2026 01:44
@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

Visual diffs detected

Pixel differences were found in visual tests. This is not blocking — reproduce locally with cd tests/visual && pnpm docs:download && pnpm test to review diffs.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d5ef184180

ℹ️ About Codex in GitHub

Codex has been enabled to automatically 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 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Copy link
Collaborator

@harbournick harbournick left a comment

Choose a reason for hiding this comment

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

LGTM

@harbournick harbournick merged commit 37bc030 into main Feb 13, 2026
4 checks passed
@harbournick harbournick deleted the andrii/sd-1890-telemetry-disable-telemetry-for-sub-editors branch February 13, 2026 18:26
@superdoc-bot
Copy link
Contributor

superdoc-bot bot commented Feb 13, 2026

🎉 This PR is included in superdoc v1.13.0-next.8

The release is available on GitHub release

superdoc-bot bot pushed a commit that referenced this pull request Feb 14, 2026
# [1.13.0](v1.12.0...v1.13.0) (2026-02-14)

### Bug Fixes

* anchor table overlaps text ([#1995](#1995)) ([fc05e29](fc05e29))
* collaboration cursor styles fix ([fd6db10](fd6db10))
* ensure we do not duplicate bubble text ([#1934](#1934)) ([c41cf9e](c41cf9e))
* ignore sdBlockId when pasting content ([#2010](#2010)) ([1b08572](1b08572))
* normalize bookmarks in tables ([#1892](#1892)) ([369b7e1](369b7e1))
* replace Node.js Buffer APIs with browser-native alternatives ([#2028](#2028)) ([b17774a](b17774a)), closes [#exportProcessMediaFiles](https://github.com/superdoc-dev/superdoc/issues/exportProcessMediaFiles)
* use correct template syntax for GitHub release URL in PR comments ([9d1bca2](9d1bca2))

### Features

* enabled telemetry by default and added documentation ([#2001](#2001)) ([8598ef7](8598ef7))
* enhance telemetry handling for sub-editors ([#2017](#2017)) ([37bc030](37bc030))
* update telemetry configuration to prioritize root licenseKey ([#2016](#2016)) ([3b4ff6b](3b4ff6b))
@superdoc-bot
Copy link
Contributor

superdoc-bot bot commented Feb 14, 2026

🎉 This PR is included in superdoc v1.13.0

The release is available on GitHub release

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.

2 participants