Skip to content

Revert "Updated selected transaction If getting new updates"#50322

Closed
cretadn22 wants to merge 1 commit into
Expensify:mainfrom
cretadn22:revert-49829-deselect-option-when-bulk-action
Closed

Revert "Updated selected transaction If getting new updates"#50322
cretadn22 wants to merge 1 commit into
Expensify:mainfrom
cretadn22:revert-49829-deselect-option-when-bulk-action

Conversation

@cretadn22

Copy link
Copy Markdown
Contributor

Reverts #49829 to Fix #50286

@cretadn22 cretadn22 requested a review from a team as a code owner October 7, 2024 03:38
@cretadn22 cretadn22 marked this pull request as draft October 7, 2024 03:38
@melvin-bot melvin-bot Bot requested a review from marcaaron October 7, 2024 03:38
@melvin-bot

melvin-bot Bot commented Oct 7, 2024

Copy link
Copy Markdown

@marcaaron Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

@melvin-bot melvin-bot Bot removed the request for review from a team October 7, 2024 03:38
@cretadn22 cretadn22 marked this pull request as ready for review October 7, 2024 03:39
@cretadn22

Copy link
Copy Markdown
Contributor Author

cc @luacmartins @jjcoffee

@jjcoffee

jjcoffee commented Oct 7, 2024

Copy link
Copy Markdown
Contributor

@cretadn22 Why the revert? Can't we find a fix?

@cretadn22

Copy link
Copy Markdown
Contributor Author

I am working on a fix, the reverted PR is created in case we need to fix asap

@thienlnam

Copy link
Copy Markdown
Contributor

@cretadn22 How is the fix coming along? I'm planning on deploying within a couple hours so if we could get something merged by then that would be ideal

@cretadn22

Copy link
Copy Markdown
Contributor Author

The fix has been implemented in this PR: #50347

@cretadn22

Copy link
Copy Markdown
Contributor Author

@thienlnam If @jjcoffee can review #50347 soon, I'll go ahead and close this PR.

@cretadn22 cretadn22 closed this Oct 7, 2024
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.

[HOLD for payment 2024-10-16] Search - Selected expense does not remain selected, gets auto deselected

3 participants