nkdengineer

Results 24 comments of nkdengineer

## Proposal ### Please re-state the problem that we are trying to solve in this issue. When opening a new DM to an existing user, draft message is lost ###...

@Santhosh-Sellavel I tested and it works well. Actually when we search for new user `SearchForReport` API return the `login` of user so we still see this user in start chat....

@Santhosh-Sellavel The test branch here https://github.com/nkdengineer/App/tree/fix/38983. If it still doesn't work, please share the video here so I can see what is the problem thanks.

@Santhosh-Sellavel the problem is here because, we save the draft with debounced 1s so if the `OpenReport` complete quickly the draft of the optimistic report isn't save and then we...

@Santhosh-Sellavel Thanks for your feedback, updated my proposal https://github.com/Expensify/App/issues/38983#issuecomment-2019570054 with new solution to make sure the draft is saved before switching.

@Santhosh-Sellavel I updated the test branch https://github.com/nkdengineer/App/tree/fix/38983, please help to check again.

## Proposal ### Please re-state the problem that we are trying to solve in this issue. We're currently using a split button, which doesn't make sense for this UI interaction...

[Proposal updated](https://github.com/Expensify/App/issues/39832#issuecomment-2043170656) to remove the additional padding, based on [design team's confirmation](https://github.com/Expensify/App/issues/39832#issuecomment-2043265797)

> This would lead to performance issues because of unnecessary re-renders. @ShridharGoel This is a trivial issue and can be easily avoided with the use of memoization (like `useCallback`). I...

## Proposal ### Please re-state the problem that we are trying to solve in this issue. Add a search bar on the workspaces list ### What is the root cause...