Skip to content

fix(config): remove notification on dnd#1152

Open
ShGKme wants to merge 2 commits into
mainfrom
fix/remove-notification-on-dnd
Open

fix(config): remove notification on dnd#1152
ShGKme wants to merge 2 commits into
mainfrom
fix/remove-notification-on-dnd

Conversation

@ShGKme

@ShGKme ShGKme commented Mar 4, 2025

Copy link
Copy Markdown
Contributor

☑️ Resolves

  • This option is not available in general case
  • Removed for consistency across clients

🖼️ Screenshots

🏚️ Before 🏡 After
image image

ShGKme added 2 commits March 3, 2025 15:26
Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
@ShGKme ShGKme added bug Something isn't working 3. to review labels Mar 4, 2025
@ShGKme ShGKme added this to the 🪴 Next Release milestone Mar 4, 2025
@ShGKme ShGKme self-assigned this Mar 4, 2025

@nickvergessen nickvergessen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could keep the non-bool values for the future, but fine either way

@Antreesy

Antreesy commented Mar 5, 2025

Copy link
Copy Markdown
Contributor

I'd also only drop 'always' for now

@ShGKme

ShGKme commented Mar 5, 2025

Copy link
Copy Markdown
Contributor Author

Boolean values reflect corresponding config values on the server
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants