[Pay now][$500] Payment for https://github.com/Expensify/App/pull/30829

[mallenexpensify]

• Video player #30829

Which is part of

• [Tracking] Add a Video Player to New Expensify #20471

[mallenexpensify]

@akinwale , can you please accept the job and reply here once you have?
https://www.upwork.com/jobs/~01d7b0f6c0865460ec

[melvin-bot]

⚠️ Could not update price automatically because there is no linked Upwork Job ID. The BZ team member will need to update the price manually in Upwork.

[akinwale]

@mallenexpensify Accepted. Thanks!

[mallenexpensify]

@akinwale can you comment on the PR here. I want to know if there's a regression and compensation should be reduce by 50%

This line caused crash - #36834
source is not always string, i.e. svg

[akinwale]

@mallenexpensify It appears that the regression came from the PR, but it's an edge case that we wouldn't have been able to catch over the course of testing the video playback feature.

[mallenexpensify]

Thanks @akinwale to close the loop, for 'edge case' you're saying it falls under this part of the C+ process doc

*“Should have” = C+ should have caught the bug by fully following the checklist. If C+ skips a step or completed the checklist incompletely, payment will be cut in half.

Is that correct? (asking cuz I want to know if we need to consider updating anything)

[akinwale]

Is that correct? (asking cuz I want to know if we need to consider updating anything)

Yes, this seems to be the case. The bug happens when viewing a workspace avatar, not with the video player itself.

[mallenexpensify]

Contributor+: @akinwale paid $500 via Upwork.