mattermost-community / mattermost-plugin-jitsi

Jitsi plugin for Mattermost :electric_plug:
Apache License 2.0
195 stars 90 forks source link

gh-165 #192

Closed maisnamrajusingh closed 3 years ago

maisnamrajusingh commented 3 years ago

Summary

fixes the spacing issue

Screenshot 2021-06-15 at 4 50 46 PM

Ticket Link

Fixes https://github.com/mattermost/mattermost-plugin-jitsi/issues/165

codecov-commenter commented 3 years ago

Codecov Report

Merging #192 (f936f51) into master (e5e3252) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #192   +/-   ##
=======================================
  Coverage   44.95%   44.95%           
=======================================
  Files           7        7           
  Lines         803      803           
=======================================
  Hits          361      361           
  Misses        417      417           
  Partials       25       25           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update e5e3252...f936f51. Read the comment docs.

jasonblais commented 3 years ago

@hanzei I think for these PRs we agreed QA testing happens prior to cutting a release, to make the QA load lighter?

hanzei commented 3 years ago

If @DHaussermann agrees with that, I'm fine with merging the PR without QA testing.

jasonblais commented 3 years ago

Yes, that is what we agreed on the call with Demansol last week, but I can confirm with Dylan as well