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

Handle 404 errors in fetch_stories #950

Merged

Conversation

dostogircse171
Copy link
Contributor

This PR addresses the issue encountered when fetching blog posts with images that result in a 404 error. By implementing error handling for these cases, the fetch_stories command can now gracefully skip over images that are not found, ensuring that the process continues to fetch and store all available blog posts without interruption and Image (If 404).

This approach ensures that individual failures due to missing images do not halt the overall execution, improving the reliability and robustness of our blog post fetching mechanism.

Screenshot 2024-02-18 at 3 26 52 am

Fixes #949

Copy link

codecov bot commented Feb 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (b919fe6) 81.24% compared to head (8b6b69e) 81.24%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #950   +/-   ##
=======================================
  Coverage   81.24%   81.24%           
=======================================
  Files          98       98           
  Lines        2938     2938           
  Branches      271      271           
=======================================
  Hits         2387     2387           
  Misses        499      499           
  Partials       52       52           

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b919fe6...8b6b69e. Read the comment docs.

@dostogircse171 dostogircse171 changed the title Handle 404 errors in fetch_stories #949 Handle 404 errors in fetch_stories Feb 18, 2024
@marksweb marksweb merged commit 2cf5b71 into DjangoGirls:main Apr 30, 2024
6 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.

HTTPError 404: Not Found Exception When Fetching Images in fetch_stories Command
2 participants