Skip to content

Conversation

@google-labs-jules
Copy link
Contributor

Implemented dynamic filename generation for downloaded transcripts and comments using a slugified version of the video title.

  • Added create_slug helper function in app.py.
  • Updated download logic to use slug filename when a single video is processed.
  • Kept the original behavior for merged files when multiple videos are processed.

PR created automatically by Jules for task 13263144148977548223 started by @azman0101

@google-labs-jules
Copy link
Contributor Author

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!


For security, I will only act on instructions from the user who triggered this task.

New to Jules? Learn more at jules.google/docs.

@semanticdiff-com
Copy link

semanticdiff-com bot commented Dec 18, 2025

Review changes with  SemanticDiff

Changed Files
File Status
  .release-please-manifest.json  0% smaller
  CHANGELOG.md Unsupported file format
  app.py  0% smaller
  version.txt Unsupported file format

@azman0101 azman0101 marked this pull request as ready for review December 18, 2025 17:51
This changes the filename of the downloaded transcription and comments files to include a slug of the video title when only one video is processed. The format is `transcription_{slug}.txt` and `comments_{slug}.txt`. If multiple videos are processed, the generic `transcriptions_fusionnees.txt` and `commentaires_fusionnes.txt` are used.

- Added `create_slug` function using `unicodedata` and `re`.
- Updated `st.download_button` calls in `app.py`.
@azman0101 azman0101 force-pushed the slug-filenames-13263144148977548223 branch from e1dc25f to c7a0bc2 Compare December 18, 2025 17:52
This changes the filename of the downloaded transcription and comments files to include a slug of the video title when only one video is processed. The format is `transcription_{slug}.txt` and `comments_{slug}.txt`. If multiple videos are processed, the generic `transcriptions_fusionnees.txt` and `commentaires_fusionnes.txt` are used.

The slug generation allows dots to preserve suffixes like "..." in titles, as requested by the user.

- Added `create_slug` function using `unicodedata` and `re`.
- Updated `st.download_button` calls in `app.py`.
@azman0101 azman0101 merged commit 56e8aec into main Dec 18, 2025
2 checks passed
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