Skip to content

[HOLD for payment 2024-09-19][$1000] Replace report.notificationPreference with report.participants #46835

@puneetlath

Description

@puneetlath

Right now we are sending duplicate data in the report participants object and the notificationPreference object. To unify the data in one place and simplify the back-end flow, we are going to update the notificationPreference to be a part of the participants object.

{
    [accountID1]: {role: 'admin', notificationPreference: 'mute'},
    [accountID2]: {role: 'member', notificationPreference: 'always'},
}

To make this update we will need to:

  • Update back-end to start including notificationPreference in the participants object whenever it is currently sent https://github.com/Expensify/Auth/pull/11920
  • Update back-end to start including full participants object whenever sending the report
  • Re-add the Web tests that were commented out in https://github.com/Expensify/Web-Expensify/pull/43027
  • Update the front-end to start using this notificationPreference instead of the report.notificationPreference or report.participants.hidden values
  • Update the back-end to stop sending the deprecated fields

cc @shubham1206agra

Issue OwnerCurrent Issue Owner: @puneetlath

Metadata

Metadata

Labels

Awaiting PaymentAuto-added when associated PR is deployed to productionNewFeatureSomething to build that is a new item.WeeklyKSv2

Type

No type

Projects

Status

Done

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions