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

Fix error appearing when selectedKeys contain invalid key #244

Merged
merged 2 commits into from
Nov 3, 2023

Conversation

Walingar
Copy link
Collaborator

@Walingar Walingar commented Nov 3, 2023

Selected item keys for items that don't exist anymore don't get purged when the backing data changes, causing stale keys to remain in the map. Pending a more comprehensive cleanup as detailed in #242, this is a work-around that is good enough for now. It is not a long-term solution, to be clear.

In this case, we should just skip invalid keys when receiving their indices
@Walingar Walingar requested a review from rock3r November 3, 2023 15:15
Copy link
Collaborator

@rock3r rock3r left a comment

Choose a reason for hiding this comment

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

LGTM!

@rock3r rock3r enabled auto-merge (squash) November 3, 2023 15:18
@rock3r rock3r added the bug Something isn't working label Nov 3, 2023
@rock3r rock3r merged commit d105d9e into main Nov 3, 2023
10 checks passed
@rock3r rock3r deleted the nr/non-existing-key-selectable-lazy-column branch November 3, 2023 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants