Skip to content

OUT-2140 | Reassignment is not possible - #911

Merged
arpandhakal merged 1 commit into
feature/M15from
OUT-2146
Aug 8, 2025
Merged

OUT-2140 | Reassignment is not possible#911
arpandhakal merged 1 commit into
feature/M15from
OUT-2146

Conversation

@arpandhakal

@arpandhakal arpandhakal commented Aug 8, 2025

Copy link
Copy Markdown
Collaborator

Changes

  • closed the popper and called the update(reassignment) function if some value is selected.
  • In case of value being removed(unassigned), prevented calling the update function and closing the popper immediately, instead listened to the close state of popper and then called the update function when popper is closed and no assignee is selected.

Testing Criteria

…lot selector update.

- closed the popper and called the update(reassignment) function if some value is selected.
- In case of value being removed(unassigned), prevented calling the update function and closing the popper immediately, instead listened to the close state of popper and then called the update function when popper is closed and no assignee is selected.
@arpandhakal arpandhakal self-assigned this Aug 8, 2025
@linear

linear Bot commented Aug 8, 2025

Copy link
Copy Markdown

@vercel

vercel Bot commented Aug 8, 2025

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
tasks-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 8, 2025 10:08am

@arpandhakal arpandhakal changed the title OUT-2146 | Reassignment is not possible OUT-2140 | Reassignment is not possible Aug 8, 2025
@linear

linear Bot commented Aug 8, 2025

Copy link
Copy Markdown

@SandipBajracharya SandipBajracharya left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM 🙌

@arpandhakal
arpandhakal merged commit 9e3ded6 into feature/M15 Aug 8, 2025
3 checks passed
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