Skip to content

[YouTube Downloader] Avoid to run onSubmit while fetching video#17641

Merged
raycastbot merged 2 commits intoraycast:mainfrom
LitoMore:yt-dlp/fixes
Mar 7, 2025
Merged

[YouTube Downloader] Avoid to run onSubmit while fetching video#17641
raycastbot merged 2 commits intoraycast:mainfrom
LitoMore:yt-dlp/fixes

Conversation

@LitoMore
Copy link
Contributor

@LitoMore LitoMore commented Mar 7, 2025

Description

Closes #17625

  • Avoid to run onSubmit while fetching video

Checklist

@raycastbot raycastbot added extension fix / improvement Label for PRs with extension's fix improvements extension: youtube-downloader Issues related to the youtube-downloader extension OP is contributor The OP of the PR is a contributor of the extension labels Mar 7, 2025
@raycastbot
Copy link
Collaborator

raycastbot commented Mar 7, 2025

Thank you for your contribution! 🎉

🔔 @vimtor @pernielsentikaer @darmiel @ridemountainpig @remorses @anwarulislam you might want to have a look.

You can use this guide to learn how to check out the Pull Request locally in order to test it.

Due to our current reduced availability, the initial review may take up to 10-15 business days

Copy link
Collaborator

@pernielsentikaer pernielsentikaer left a comment

Choose a reason for hiding this comment

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

Hi 👋

Looks good to me, approved 🔥

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

This PR adds validation checks to prevent form submission while fetching video data and ensures a format is selected before downloading in the YouTube Downloader extension.

  • Added format validation check in extensions/youtube-downloader/src/index.tsx to prevent downloads without a selected format
  • Added new changelog entry at the top of CHANGELOG.md with proper {PR_MERGE_DATE} template
  • The onSubmit handler now properly handles the loading state to prevent duplicate submissions
  • Consider using showFailureToast from @raycast/utils instead of manual showToast with failure style in error handling

2 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile

@raycastbot raycastbot merged commit 50372f9 into raycast:main Mar 7, 2025
2 checks passed
@LitoMore LitoMore deleted the yt-dlp/fixes branch March 7, 2025 14:26
@github-actions
Copy link
Contributor

github-actions bot commented Mar 7, 2025

Published to the Raycast Store:
https://raycast.com/vimtor/youtube-downloader

@raycastbot
Copy link
Collaborator

🎉 🎉 🎉

We've rewarded your Raycast account with some credits. You will soon be able to exchange them for some swag.

@LitoMore LitoMore changed the title [YouTube Downloader] Avoid to run run onSubmit while fetching video [YouTube Downloader] Avoid to run onSubmit while fetching video Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

extension fix / improvement Label for PRs with extension's fix improvements extension: youtube-downloader Issues related to the youtube-downloader extension OP is contributor The OP of the PR is a contributor of the extension

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Video Downloader] ...

3 participants