Skip to content

fix(power): fix scheduled shutdown notifications - #161

Merged
deepin-bot[bot] merged 1 commit into
linuxdeepin:masterfrom
mhduiy:task/x11power
Sep 4, 2026
Merged

fix(power): fix scheduled shutdown notifications#161
deepin-bot[bot] merged 1 commit into
linuxdeepin:masterfrom
mhduiy:task/x11power

Conversation

@mhduiy

@mhduiy mhduiy commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

修复定时关机通知音效及工作日定时关机流程。

  • 恢复旧版通知动作键,避免通知音效被错误抑制
  • 修复工作日信息解析及日历 D-Bus 参数类型

PMS: BUG-375825
PMS: BUG-375829

Summary by Sourcery

Fix scheduled shutdown notifications and working-day scheduling.

Bug Fixes:

  • Restore scheduled-shutdown notification action keys so notification sounds and cancel/shutdown actions work correctly.
  • Fix workday calendar parsing and calendar D-Bus argument types so scheduled shutdowns follow the correct working-day schedule.

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Sorry @mhduiy, your pull request is larger than the review limit of 150,000 diff characters

1. Restore legacy notification action keys to keep notification sounds enabled.
2. Parse calendar service fields using the actual lowercase D-Bus response names.
3. Use uint32-compatible arguments for the calendar D-Bus method.

Log: Fix scheduled shutdown notification sound and workday scheduling.
Influence: Scheduled shutdown works correctly on working days.

fix(power): 修复定时关机通知问题

1. 恢复旧版通知动作键,避免通知音效被错误抑制。
2. 按日历服务实际返回的字段名解析工作日信息。
3. 使用与日历 D-Bus 方法兼容的 uint32 参数类型。

Log: 修复定时关机通知音效和工作日定时关机调度问题。
PMS: BUG-375825
PMS: BUG-375829
Influence: 工作日定时关机和通知音效恢复正常。
Change-Id: I8cb56dcb8b0e0ec9c2a71977087356e2b7692777
@deepin-ci-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: fly602, mhduiy

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mhduiy

mhduiy commented Sep 4, 2026

Copy link
Copy Markdown
Contributor Author

/forcemerge

@deepin-bot

deepin-bot Bot commented Sep 4, 2026

Copy link
Copy Markdown

This pr force merged! (status: blocked)

@deepin-bot
deepin-bot Bot merged commit 1b5a8bd into linuxdeepin:master Sep 4, 2026
5 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants