Skip to content

Conversation

@jog1t
Copy link
Contributor

@jog1t jog1t commented Jun 13, 2025

Changes

@graphite-app
Copy link
Contributor

graphite-app bot commented Jun 13, 2025

How to use the Graphite Merge Queue

Add the label merge-queue to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

@jog1t jog1t merged commit 117f5dd into main Jun 13, 2025
0 of 3 checks passed
@jog1t jog1t deleted the fix/invalid-atom-value branch June 13, 2025 21:52
Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

Restructures actor filter atom implementation to fix invalid atom value handling in the Jotai state management system.

  • Modifies actorsInternalFilterAtom in frontend/packages/components/src/actors/actor-context.tsx to store filter function in an object structure with fn property
  • Updates frontend/apps/hub/src/domains/project/components/actors/actors-provider.tsx to properly wrap internal filter function in object format when setting atom value
  • Improves type safety and state handling for actor filtering logic in the Hub application

2 files reviewed, no comments
Edit PR Review Bot Settings | Greptile

@graphite-app
Copy link
Contributor

graphite-app bot commented Jun 13, 2025

Graphite Automations

"Test" took an action on this PR • (06/13/25)

1 assignee was added to this PR based on Kacper Wojciechowski's automation.

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.

2 participants