Skip to content

[ReviewEntriesCompleted] Fix key error#4042

Merged
imnasnainaec merged 1 commit intomasterfrom
review-entries-completed-deleted
Nov 25, 2025
Merged

[ReviewEntriesCompleted] Fix key error#4042
imnasnainaec merged 1 commit intomasterfrom
review-entries-completed-deleted

Conversation

@imnasnainaec
Copy link
Collaborator

@imnasnainaec imnasnainaec commented Nov 24, 2025

Follow-up to #3956


This change is Reviewable

@imnasnainaec imnasnainaec self-assigned this Nov 24, 2025
@imnasnainaec imnasnainaec added 🟥High High-priority PR: please review this asap! bug Something isn't working labels Nov 24, 2025
@codecov
Copy link

codecov bot commented Nov 24, 2025

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 74.66%. Comparing base (a49f082) to head (71532f1).
⚠️ Report is 25 commits behind head on master.

Files with missing lines Patch % Lines
src/goals/ReviewEntries/ReviewEntriesCompleted.tsx 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4042   +/-   ##
=======================================
  Coverage   74.66%   74.66%           
=======================================
  Files         293      293           
  Lines       10890    10890           
  Branches     1364     1364           
=======================================
  Hits         8131     8131           
  Misses       2362     2362           
  Partials      397      397           
Flag Coverage Δ
backend 85.66% <ø> (ø)
frontend 66.11% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Collaborator Author

@imnasnainaec imnasnainaec left a comment

Choose a reason for hiding this comment

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

@imnasnainaec reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @imnasnainaec)

@imnasnainaec imnasnainaec merged commit c7a2fc2 into master Nov 25, 2025
19 checks passed
@imnasnainaec imnasnainaec deleted the review-entries-completed-deleted branch November 25, 2025 01:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working frontend goal: ReviewEntries 🟥High High-priority PR: please review this asap!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant