[MM-53988] Improve logic for showing recording stopped banner

[streamer45]

Summary

PR fixes an issues that would cause the Recording has stopped. Processing... message to appear to a newly changed host regardless of when the recording ended.

I am adding a threshold (currently one minute) after which we wouldn't show this. Technically we could just show it to the host at the time of stopping the recording but felt it may be useful to keep it around in case of a sudden host change. Overall though I don't have a strong opinion myself and feel it's more of a UX decision.

Regardless of what we do here, at some point (likely as part of host controls) we may want to design a message to inform a user that they became the host.

/cc @matthewbirtch in case you'd like to chime in on any of the above.

Ticket Link

https://mattermost.atlassian.net/browse/MM-53988

[codecov-commenter]

Codecov Report

Patch coverage has no change and project coverage change: -0.08% ⚠️

Comparison is base (5c059da) 5.62% compared to head (65296e0) 5.55%.

Additional details and impacted files

@@           Coverage Diff            @@
##            main    #491      +/-   ##
========================================
- Coverage   5.62%   5.55%   -0.08%     
========================================
  Files         21      21              
  Lines       4178    4178              
========================================
- Hits         235     232       -3     
- Misses      3926    3928       +2     
- Partials      17      18       +1     

see 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[matthewbirtch]

I am adding a threshold (currently one minute) after which we wouldn't show this. Technically we could just show it to the host at the time of stopping the recording but felt it may be useful to keep it around in case of a sudden host change. Overall though I don't have a strong opinion myself and feel it's more of a UX decision.

To be honest, I'm not sure if this is something we need to do. It doesn't seem all that important to me. I'd say leave it out.

Regardless of what we do here, at some point (likely as part of host controls) we may want to design a message to inform a user that they became the host.

I agree that we will want to do this as part of host controls effort (although I dont' know if/when that will get roadmapped).

[cpoile]

Thanks!