Skip to content

Fix React warning in recent decision summary - #1002

Merged
taobojlen merged 4 commits into
mainfrom
fix/recent-decision-summary-keys
Sep 1, 2026
Merged

Fix React warning in recent decision summary#1002
taobojlen merged 4 commits into
mainfrom
fix/recent-decision-summary-keys

Conversation

@taobojlen

@taobojlen taobojlen commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Context & Requests for Reviewers

This fixes a React warning about a missing key.

Tests

Manually tested.

(Optional) Rollout Plan

N/A

Checklist

Only check items that apply to this PR; leave the rest unchecked.

  • If you changed anything user-facing (i.e. user interface or APIs):
    Did you update the CHANGELOG.md and related docs?

  • If you changed server/models/**/{ContentTypeModel,ActionModel,RuleModel,PolicyModel}.ts:
    Did you update the corresponding history tables and their triggers?

  • If you changed db/src/scripts/** and used CREATE TABLE, ADD COLUMN, or ALTER COLUMN:
    Are as many columns marked NOT NULL as possible? If some columns can sometimes be null depending on other columns, are there CHECK constraints capturing those relationships, and are these also reflected using unions in the associated Kysely types?

  • If you added a new signal in server/services/signalsService/signals/**:
    Did you classify every error case as a permanent error (SignalPermanentError, no retry) or a normal error (retryable)? Any case where the signal can't determine a score should be a SignalPermanentError.

@coderabbitai

coderabbitai Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

Next included review available in 32 minutes.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: 70206d78-0baa-4c22-9dda-c4c670b9be3a

📥 Commits

Reviewing files that changed from the base of the PR and between 05e01ae and 41e723e.

📒 Files selected for processing (1)
  • client/src/webpages/dashboard/mrt/ManualReviewRecentDecisionSummary.tsx

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@taobojlen
taobojlen force-pushed the fix/investigation-policy-select-value branch from c5debae to 32c1c05 Compare August 12, 2026 03:05
@taobojlen
taobojlen force-pushed the fix/recent-decision-summary-keys branch from 8708551 to 38de790 Compare August 12, 2026 03:05
@taobojlen
taobojlen force-pushed the fix/investigation-policy-select-value branch from 32c1c05 to 538d54e Compare August 12, 2026 03:42
@taobojlen
taobojlen force-pushed the fix/recent-decision-summary-keys branch from 38de790 to 21e5ce3 Compare August 12, 2026 03:42
@taobojlen
taobojlen force-pushed the fix/recent-decision-summary-keys branch from 21e5ce3 to 5fbfe53 Compare August 17, 2026 10:22
@taobojlen
taobojlen force-pushed the fix/investigation-policy-select-value branch from 538d54e to 2da1d55 Compare August 17, 2026 10:22
@taobojlen
taobojlen changed the base branch from fix/investigation-policy-select-value to main August 17, 2026 10:23
@taobojlen taobojlen changed the title Fix recent decision summary keys Fix React warning in recent decision summary Aug 18, 2026
@taobojlen
taobojlen marked this pull request as ready for review August 18, 2026 13:06

@juanmrad juanmrad left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍🏻

@cassidyjames
cassidyjames requested a review from a team as a code owner August 31, 2026 22:40
@cassidyjames
cassidyjames enabled auto-merge (squash) August 31, 2026 22:40
@cassidyjames
cassidyjames disabled auto-merge August 31, 2026 23:32
@taobojlen
taobojlen merged commit 9477728 into main Sep 1, 2026
18 of 19 checks passed
@taobojlen
taobojlen deleted the fix/recent-decision-summary-keys branch September 1, 2026 08:23
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.

3 participants