Skip to content

Add support for storing transcripts#112

Merged
tisnik merged 1 commit into
lightspeed-core:mainfrom
umago:transcripts
Jun 18, 2025
Merged

Add support for storing transcripts#112
tisnik merged 1 commit into
lightspeed-core:mainfrom
umago:transcripts

Conversation

@umago
Copy link
Copy Markdown
Contributor

@umago umago commented Jun 18, 2025

Description

This patch adds support for storing transcripts in lightspeed-core. By default, transcripts are disabled.

The transcripts maintain the same format as they were implemented in road-core so the user data collector and existing pipeline should work seamlessly as before.

Type of change

  • Refactor
  • New feature
  • Bug fix
  • CVE fix
  • Optimization
  • Documentation Update
  • Configuration Update
  • Bump-up service version
  • Bump-up dependent library
  • Bump-up library or tool used for development (does not change the final image)
  • CI configuration change
  • Konflux configuration change
  • Unit tests improvement
  • Integration tests improvement
  • End to end tests improvement

Related Tickets & Documents

  • Related Issue #LCORE-214
  • Closes #

Checklist before requesting a review

  • I have performed a self-review of my code.
  • PR has passed all pre-merge test jobs.
  • If it is a core feature, I have added thorough tests.

Testing

  • Run unit-tests

This patch adds support for storing transcripts in lightspeed-core. By
default, transcripts are disabled.

The transcripts maintain the same format as they were implemented in
road-core so the user data collector and existing pipeline should work
seamlessly as before.

Signed-off-by: Lucas Alvares Gomes <lucasagomes@gmail.com>
Copy link
Copy Markdown
Contributor

@tisnik tisnik left a comment

Choose a reason for hiding this comment

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

it looks ok (we just need to think about better name than common, but it's outside scope of this PR)

@tisnik tisnik merged commit 0565cb7 into lightspeed-core:main Jun 18, 2025
18 checks passed
@umago umago deleted the transcripts branch June 20, 2025 10:20
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.

2 participants