[HOLD for payment 2023-12-29] [CRITICAL] Update HTMLEngineProvider to use VideoRenderer

[francoisl]

Part of the NewDot Video Player project

Main issue: #20471
Doc section: https://docs.google.com/document/d/1Fh5Nu3D0-VW7xuV9Qc-OWWZl3-W5azx7Rr1lzNw4UGo/edit#heading=h.3iq5gzws08j5
Project: https://github.com/orgs/Expensify/projects/109

Feature Description

Update HTMLEngineProvider so we can render the video component on messages whose html value is a <video> tag

cc @Skalakid

[melvin-bot]

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

[Skalakid]

Hi I’m Michał from Software Mansion, an expert agency, and I’d like to work on this issue

[francoisl]

@Skalakid this is the first part that would not require all the components to be implemented, right? I'm trying to get a sense of whether this is something we can aim to finish next week, for the weekly update I'm preparing for tomorrow.

[Skalakid]

@francoisl No video player component need to be implemented to update HTMLEngineProvider. I think we can start by rendering <video> tags like current <a> tags. I created draft PR with the required changes

[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.15-5 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• Update HTMLEngineProvider to use VideoRenderer #33145

If no regressions arise, payment will be issued on 2023-12-29. 🎊

After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

• External issue reporter
• Contributor that fixed the issue
• Contributor+ that helped on the issue and/or PR

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

• @Skalakid 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:

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

[francoisl]

This is on production, closing the issue.

[Ollyws]

@mallenexpensify I just noticed I was never paid for my review of #33145, any chance you could process payment for that? Thanks!

[mallenexpensify]

Wowza, sorry @Ollyws . can you please accept the job and reply here once you have?
https://www.upwork.com/jobs/~013ee0d8a7106d964d

[Ollyws]

@mallenexpensify accepted, thanks!

[mallenexpensify]

Contributor+: @Ollyws paid $500 via Upwork.

Sorry again for missing this.