Skip to content

Commit

Permalink
Merge pull request #19565 from calixteman/signature_fix_telemetry
Browse files Browse the repository at this point in the history
[Editor] Fix the telemetry for the signature stuff
  • Loading branch information
calixteman authored Feb 27, 2025
2 parents da17c7b + bbf9bfc commit a4fea2d
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions web/signature_manager.js
Original file line number Diff line number Diff line change
Expand Up @@ -177,6 +177,7 @@ class SignatureManager {
"click",
() => {
this.#reportTelemetry({
type: "signature",
action: "pdfjs.signature.clear",
data: {
type: this.#currentTab,
Expand Down Expand Up @@ -738,6 +739,7 @@ class SignatureManager {
if (await this.#signatureStorage.delete(uuid)) {
div.remove();
this.#reportTelemetry({
type: "signature",
action: "pdfjs.signature.delete_saved",
data: {
savedCount: await this.#signatureStorage.size(),
Expand Down Expand Up @@ -924,6 +926,7 @@ class SignatureManager {

const altText = this.#tabsToAltText.get(type);
this.#reportTelemetry({
type: "signature",
action: "pdfjs.signature.created",
data: {
type,
Expand Down

0 comments on commit a4fea2d

Please sign in to comment.