Skip to content

LT-22314: Add filename to media line#772

Merged
aror92 merged 1 commit intomainfrom
LT-22314-addfilename
Mar 18, 2026
Merged

LT-22314: Add filename to media line#772
aror92 merged 1 commit intomainfrom
LT-22314-addfilename

Conversation

@aror92
Copy link
Copy Markdown
Contributor

@aror92 aror92 commented Mar 18, 2026

  • Add "File: <filename>" to media line after the speaker info
  • Add "File: <filename>" after the media not found message
  • Update speaker name handling to get the name associated with the specified writing system, using the default stored in ChooserNameTS as a backup.

This change is Reviewable

* Add "File: <filename>" to media line after the speaker info
* Add "File: <filename>" after the media not found message
* Update speaker name handling to get the name associated
with the specified writing system, using the default
stored in ChooserNameTS as a backup.

Change-Id: Ifee9d92b8991491bb1b6eef74b662ceca9731a74
@github-actions
Copy link
Copy Markdown

NUnit Tests

    1 files  ±0      1 suites  ±0   6m 16s ⏱️ +13s
4 074 tests ±0  4 003 ✅ ±0  71 💤 ±0  0 ❌ ±0 
4 083 runs  ±0  4 012 ✅ ±0  71 💤 ±0  0 ❌ ±0 

Results for commit e483b25. ± Comparison against base commit 564ccbb.

Copy link
Copy Markdown
Contributor

@mark-sil mark-sil left a comment

Choose a reason for hiding this comment

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

@mark-sil reviewed 3 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on aror92).

Copy link
Copy Markdown
Contributor

@mark-sil mark-sil left a comment

Choose a reason for hiding this comment

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

Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on aror92).

@aror92 aror92 merged commit 2fd0af8 into main Mar 18, 2026
7 checks passed
@aror92 aror92 deleted the LT-22314-addfilename branch March 18, 2026 17:22
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