OpenReport API returns empty lastActorAccountID when the last reportAction is ACTIONABLEJOINREQUEST type

[hoangzinh]

Original GH:
#51069

Action Performed:

1. Log in as user A
2. Go to workspace settings
3. Click "share" and copy the URL
4. Open the URL and login with different account user B
5. Log-in back to user A and go to #admin room of the workspace
6. Reload the report screen

Expected Result:

In LHN, it should show "Expensify: {userB email} requested to join the workspace"

Actual Result:

In LHN, it's showing "You: {userB email} requested to join the workspace"

Screenshots/Videos

Add any screenshot/video evidence

Please checkout #58897 (comment)

[melvin-bot]

Triggered auto assignment to @yuwenmemon (AutoAssignerNewDotQuality)

[hoangzinh]

cc @AndrewGable ^

[yuwenmemon]

I can look into this

[melvin-bot]

@yuwenmemon Eep! 4 days overdue now. Issues have feelings too...

[hoangzinh]

How is this issue going? @yuwenmemon @AndrewGable

[yuwenmemon]

@hoangzinh Are you sure these steps are right? I'm not able to reproduce the issue when following them. This is what I see in the #admins room:

[hoangzinh]

Oh, I realize I faced a different bug; LHN now displays the previous message after step 6. cc @nkdengineer if you face same issue as me or @yuwenmemon

Screen.Recording.2025-04-23.at.18.29.18.mov

[melvin-bot]

@yuwenmemon Still overdue 6 days?! Let's take care of this!

[hoangzinh]

I tried again today and have same result #60154 (comment). cc @nkdengineer @yuwenmemon

[hoangzinh]

I feel we can continue the process with your PR @nkdengineer. Because LHN now displays the previous message, it seems like a different bug. Wdyt?

[nkdengineer]

Sorry for the delay here. It's very late in my timezone now. I'll give an update tommorrow

[yuwenmemon]

Sounds good @nkdengineer let me know if you think anything needs to be done on my end.

[nkdengineer]

I feel we can continue the process with your PR @nkdengineer. Because LHN now displays the previous message, it seems like a different bug. Wdyt?

@yuwenmemon @hoangzinh I can't reproduce the bug in the OP and completely agree with @hoangzinh here

[melvin-bot]

@yuwenmemon Huh... This is 4 days overdue. Who can take care of this?

[yuwenmemon]

@nkdengineer so what does that mean? What are the next steps as you see it?

[nkdengineer]

I feel we can continue the process with your PR @nkdengineer. Because LHN now displays the previous message, it seems like a different bug. Wdyt?

As @hoangzinh mentioned here, this is a different bug. Then we'll continue the process in my previous PR @yuwenmemon

[melvin-bot]

@yuwenmemon Eep! 4 days overdue now. Issues have feelings too...

[hoangzinh]

Hi team, I'm going to close this issue as it seems we don't need it anymore. Feel free to reopen it if you don't think so.