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

Dearrow: check for 204 error instead of 404 #5259

Merged

Conversation

ChunkyProgrammer
Copy link
Member

Dearrow: check for 204 error instead of 404

Pull Request Type

  • Bugfix

Related issue

closes #5031

Description

Dearrow switched from returning error code 400 to 204 here: ajayyy/DeArrowThumbnailCache@6d6ffab

Testing

  • turn dearrow on
  • search something (ex: FreeTube)
  • Notice how there's no more broken thumbnails

Desktop

  • OS: Linux Mint
  • OS Version: 21.3
  • FreeTube version: latest nightly

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) June 12, 2024 20:50
@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Jun 12, 2024
@FreeTubeBot FreeTubeBot merged commit d56731e into FreeTubeApp:development Jun 12, 2024
5 checks passed
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Jun 12, 2024
PikachuEXE added a commit to PikachuEXE/FreeTube that referenced this pull request Jun 13, 2024
* development: (60 commits)
  check for 204 error (FreeTubeApp#5259)
  Translated using Weblate (Turkish)
  ! Fix error when fetching deleted comment replies in local API (FreeTubeApp#5255)
  Optical enhancement - Improved spacing on the about page FreeTubeApp#5210 (FreeTubeApp#5257)
  Translated using Weblate (Czech)
  Translated using Weblate (Hungarian)
  Translated using Weblate (Estonian)
  Translated using Weblate (German)
  Update playlist page to add remove duplicate videos button for user playlists (FreeTubeApp#5191)
  Respect playlist sort order in `watch-video-playlist` (FreeTubeApp#5013)
  Translated using Weblate (Serbian)
  Translated using Weblate (Chinese (Simplified))
  Translated using Weblate (Chinese (Traditional))
  Translated using Weblate (Arabic)
  Translated using Weblate (Polish)
  Translated using Weblate (Italian)
  Translated using Weblate (Turkish)
  Translated using Weblate (Croatian)
  Translated using Weblate (Dutch)
  Translated using Weblate (Spanish)
  ...
@ChunkyProgrammer ChunkyProgrammer deleted the check-for-204-error branch October 8, 2024 02:51
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.

[Bug]: Broken Thumbnails
5 participants