Skip to content

fix(api): return 404 for non-existent report updates#2012

Merged
dipexplorer merged 1 commit into
RatLoopz:mainfrom
Neverask1121:fix-report-status
Jun 17, 2026
Merged

fix(api): return 404 for non-existent report updates#2012
dipexplorer merged 1 commit into
RatLoopz:mainfrom
Neverask1121:fix-report-status

Conversation

@Neverask1121

Copy link
Copy Markdown
Contributor

🛑 STOP: Assignment & File Scope Check

  • I am assigned to this issue.
  • I verified that this PR ONLY touches the required files.

📋 PR Summary & Link

📸 Proof of Work (Screenshots / Logs)

image

Test Case 1: Existing Report

  • Updated an existing report successfully.
  • Received a successful response with HTTP 200.

Test Case 2: Non-Existent Report

  • Attempted to update a report using an invalid/non-existent report ID.
  • Verified that the API now returns HTTP 404 with the message:
{
  "error": "Report not found"
}

🏷️ PR Type

  • 🐛 type: bug
  • type: feature
  • 📖 type: docs
  • 🧪 type: testing
  • 🔒 type: security
  • type: performance
  • 🎨 type: design
  • ♻️ type: refactor
  • 🛠️ type: devops
  • type: accessibility

✅ Checklist

  • My PR has a linked issue (Closes #2011 )
  • I have pulled the latest main and resolved any conflicts

@Neverask1121

Copy link
Copy Markdown
Contributor Author

@dipexplorer Hello there, this is enough to merge and close the issue!
Because only some changes made the problem would be critical...

@dipexplorer dipexplorer added level:beginner 20 pts quality:clean multiplier x1.2 labels Jun 17, 2026
@dipexplorer dipexplorer merged commit b9925ac into RatLoopz:main Jun 17, 2026
20 of 22 checks passed
@github-project-automation github-project-automation Bot moved this from 📥 Backlog to 🎉 Merged in SahiDawa Workflow Jun 17, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🎉 Congratulations @Neverask1121! Your Pull Request "fix(api): return 404 for non-existent report updates" has been successfully merged by @dipexplorer.

Thank you for your valuable contribution to SahiDawa! 🇮🇳
If this was for GSSoC 2026, your work is officially merged and valid. Keep up the great work and feel free to claim other open issues. 🚀

Follow us on LinkedIn: https://www.linkedin.com/company/ratloopz/ to get shoutout

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gssoc:approved Approved for gssoc level:beginner 20 pts quality:clean multiplier x1.2 type:bug Something isn't working

Projects

Status: 🎉 Merged

Development

Successfully merging this pull request may close these issues.

[BUG] updateReportStatus returns 500 instead of 404 for non-existent report IDs

2 participants