[Epic]: DCR for Apps - Discussion

[jamesgorrie]

Overview

This quarter (Jan-Mar 2024) we’ll be working on implementing discussions in DCAR. We hope will reduce the amount of platform-specific logic in favour of a unified approach. We may also use this as an opportunity to simpliying existing the code and fix bugs.

Key changes required

Discussion in web and Apps differs in that in Apps post endpoints interact with the native layer directly as that's where the user information is know. e.g. postComment or reportAbuse

There has been some work to account for this by using callbacks on the CommentContainer.

When migrating discussion-rendering we had opted for expediency to not always follow this pattern, so will need to find those interactions that speak to the discussion API directly and port them over to the paradigm above.

We might need to work with Apps here to help create any endpoints that don't exist.

Step 1 - Beta release

Beta Give feedback

1. [Epic] Support mobile articles with discussion #9667
   3 of 6
   Epic +1
   
2. [Epic] Descendants of DiscussionContainer are controlled components #10056
   8 of 8
   Epic +1
3. Decide how requests to the discussion API are handled #10071
   help wanted question +2
4. Investigate read-only beta release #10565
   Discussion +1
   
5. Update the filter in MAPI to support Discussion in beta #10417
   +0
   

Step 2 - Beta or PROD release

Beta Give feedback

1. Use Context For dispatch In Discussion #10548
   Discussion +1
   
2. [Epic] Refactor discussion to use Bridget API on apps #9869
   3 of 3
   Discussion Epic +2
   
3. [EPIC] Darkmode for discussion #10331
   13 of 13
   Discussion Epic +2
4. Make sure we have consent when using storage for discussion in apps #10735
   Discussion +1
   
5. Decide how preferences are saved (order by, number of comments, threaded, mutes) #10288
   Discussion +1
6. Decide How To Manage Form State In A Performant Way #10551
   Discussion +1
7. Add useEffectReducer To Discussion #10549
   Discussion +1
   
8. Mute functionality on device

Step 3 - PROD release

Beta Give feedback

1. Update the filter in MAPI to support read only Discussion in PROD #10734
   Discussion +1
   

Unprioritised / other

Beta Give feedback

1. Remove the isWeb condition across layouts #10416
   +0
   

[mxdvl]

Rough tasks that need addressing

• How does sign-in works with Bridget?
• How do fetch requests work in WebViews?
• How is saving preferences handled?
• How should discussion behave when offline?