Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[REFACTORING DRAFT]Telemetry #1105

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

goneri
Copy link
Contributor

@goneri goneri commented Jun 8, 2024

Push events when:

  • we preare an explanation for a playbook
  • or we generate a new one

Also:

  • rename PlaybookOutlineFeedback as PlaybookGenerationFeedback and
    accept the wizardId parameter.
  • rename ContentMatches._write_to_segment() to
    ContentMatches.write_to_segment() to match the other class of the module.

@goneri goneri marked this pull request as draft June 8, 2024 00:51
@goneri goneri force-pushed the goneri/Telemetry-Schema-2-for-Playbook-Exp-Gen_31 branch from 55a1332 to 7896615 Compare June 11, 2024 11:47
Push events when:

- we preare an explanation for a playbook
- or we generate a new one

Also:

- rename `PlaybookOutlineFeedback` as `PlaybookGenerationFeedback` and
  accept the `wizardId` parameter.
- rename `ContentMatches._write_to_segment()` to
  `ContentMatches.write_to_segment()` to match the other class of the module.
@goneri goneri force-pushed the goneri/Telemetry-Schema-2-for-Playbook-Exp-Gen_31 branch from 7896615 to e5a74e6 Compare June 11, 2024 11:57
@goneri goneri changed the title Telemetry Schema 2 for Playbook Exp/Gen [REFACTORING DRAFT]Telemetry Schema 2 for Playbook Exp/Gen Jun 11, 2024
@goneri goneri changed the title [REFACTORING DRAFT]Telemetry Schema 2 for Playbook Exp/Gen [REFACTORING DRAFT]Telemetry Jun 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant