[HOLD for payment 2024-01-28] Filter out reports based on the active workspace

[hayata-suenaga]

Filter out reports based on the active workspace with the following criteria. Display only the filtered reports in LHN.

• Reports that belong to the active workspace
• DMs and GroupDMs from members on the active workspace (there is a caveat on this requirements. See this part and this part of the design doc for detailed requirements)

The list of reports that are displayed on the current LHN (we're changing the design of LHN) are generated in this part of the code. Filter out reports based on the active workspace.

When the active workspace is Expensify/All, all reports across workspaces should be displayed.

Relevant design doc section

Issue Owner

Current Issue Owner: @WojtekBoman

[melvin-bot]

Triggered auto assignment to @alexpensify (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details.

[alexpensify]

@hayata-suenaga - does this one need to remain in [Draft]?

[hayata-suenaga]

Yes because this issue is related to Wave 8 Ideal Nav, which is still in detailed design doc phase. We'll probably delegate this task to an expert agency. When it's time to pay for C+ review, I gonna assign a Bug Zero team back to this issue 🙇

[alexpensify]

Thanks for the context. I'll check in next week again to see if there is any new update.

[alexpensify]

Weekly Update: No update yet.

[hayata-suenaga]

the design doc is still in review. I gonna remove [Draft] once the review is done (which will be by the end of this week at the latest).

[alexpensify]

Weekly Update: Thank you @hayata-suenaga!

[WojtekBoman]

I'm going to work on this task

[melvin-bot]

@WojtekBoman, @hayata-suenaga Eep! 4 days overdue now. Issues have feelings too...

[hayata-suenaga]

Because of the requirement change (instead of storing the active workspace ID, we decided to store the URL of the last visited path), we need some adjustment in term of implementation for this issue.

@WojtekBoman, are you working on this right now?

[melvin-bot]

This issue has not been updated in over 15 days. @WojtekBoman, @hayata-suenaga eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

[hayata-suenaga]

The new backend API has been deployed to the staging environment. @WojtekBoman can use the new API to test their PR.

I also commented about the new API in the PR.

[mountiny]

This is completed and merged to the feature branch, great work!

[melvin-bot]

Reviewing label has been removed, please complete the "BugZero Checklist".

[melvin-bot]

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.28-0 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• [NoQA] Add policyID to getOrderedReportIDs and sortReportsByLastRead #33302

If no regressions arise, payment will be issued on 2024-01-28. 🎊

For reference, here are some details about the assignees on this issue:

• @WojtekBoman does not require payment (Contractor)

[melvin-bot]

BugZero Checklist: The PR adding this new feature has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

• [@WojtekBoman] Please propose regression test steps to ensure the new feature will work correctly on production in further releases.
• [] Link the GH issue for creating/updating the regression test once above steps have been agreed upon.

[truph01]

@hayata-suenaga I have a question related to the requirement:

DMs and GroupDMs from members on the active workspace (there is a caveat on this requirements. See this part and this part of the design doc for detailed requirements)

Here, are we displaying only the DMs, or both the DMs and their thread reports?