Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[stable22] Fix status handling #32626

Merged
merged 3 commits into from
May 31, 2022
Merged

[stable22] Fix status handling #32626

merged 3 commits into from
May 31, 2022

Commits on May 31, 2022

  1. Fix missing status with predefined status in drop-down 

    When a predefined status message was used, the status was not "processed"
so it was missing the translated message and the icon in the dropdown and the menu afterwards

Signed-off-by: Joas Schilling <coding@schilljs.com>
    @nickvergessen @skjnldsv
    nickvergessen authored and skjnldsv committed May 31, 2022
    Configuration menu
    Copy the full SHA
    22ad2fe View commit details
    Browse the repository at this point in the history

  2. Fix unit tests 

    Signed-off-by: Joas Schilling <coding@schilljs.com>
    @nickvergessen @skjnldsv
    nickvergessen authored and skjnldsv committed May 31, 2022
    Configuration menu
    Copy the full SHA
    ee0e9b7 View commit details
    Browse the repository at this point in the history

  3. Don't overwrite certain values if the refreshing comes in via short u… 

    …pdates

This e.g. happens from the talk participant list which only has the status, message and icon.
Due to the overwriting e.g. the clearAt was overwritten with null
and afterwards the status modal showed "Invalid date" as "Clear at"

Signed-off-by: Joas Schilling <coding@schilljs.com>
    @nickvergessen @skjnldsv
    nickvergessen authored and skjnldsv committed May 31, 2022
    Configuration menu
    Copy the full SHA
    e89a98b View commit details
    Browse the repository at this point in the history