Skip to content

feat: enhance login reminder with configurable settings#172

Merged
BLumia merged 1 commit intolinuxdeepin:masterfrom
fly602:master
Dec 4, 2025
Merged

feat: enhance login reminder with configurable settings#172
BLumia merged 1 commit intolinuxdeepin:masterfrom
fly602:master

Conversation

@fly602
Copy link
Copy Markdown
Contributor

@fly602 fly602 commented Dec 3, 2025

  1. Added DConfig support for LoginReminder configuration with default value false
  2. Implemented password expiration check as fallback when LoginReminder is disabled
  3. Replaced systemd service with desktop autostart file for better session integration
  4. Added translation support with multiple language files (en_US, zh_CN, zh_HK, zh_TW)
  5. Added D-Bus service watcher to wait for notification service availability
  6. Improved notification content with detailed login information and password warnings
  7. Added action button to directly open password change page in control center

Log: Added configurable login reminder feature with password expiration warnings

Influence:

  1. Test login reminder functionality with DConfig enabled and disabled states
  2. Verify password expiration notifications work independently
  3. Check translation loading for different locales
  4. Test notification action button opens control center password page
  5. Verify autostart behavior after system login
  6. Test D-Bus service dependency handling

feat: 增强登录提醒功能并支持配置设置

  1. 添加 DConfig 支持登录提醒配置,默认值为关闭
  2. 实现密码过期检查作为登录提醒禁用时的备用功能
  3. 将 systemd 服务替换为桌面自启动文件以实现更好的会话集成
  4. 添加多语言翻译支持(英文、简体中文、繁体中文香港、繁体中文台湾)
  5. 添加 D-Bus 服务监视器以等待通知服务可用
  6. 改进通知内容,包含详细的登录信息和密码警告
  7. 添加操作按钮可直接打开控制中心的密码修改页面

Log: 新增可配置的登录提醒功能,包含密码过期警告

Influence:

  1. 测试 DConfig 启用和禁用状态下的登录提醒功能
  2. 验证密码过期通知是否独立工作
  3. 检查不同语言环境的翻译加载
  4. 测试通知操作按钮是否打开控制中心密码页面
  5. 验证系统登录后的自启动行为
  6. 测试 D-Bus 服务依赖处理

Copy link
Copy Markdown

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

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

Sorry @fly602, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

Comment thread .tx/config Outdated
1. Added DConfig support for LoginReminder configuration with default
value false
2. Implemented password expiration check as fallback when LoginReminder
is disabled
3. Replaced systemd service with desktop autostart file for better
session integration
4. Added translation support with multiple language files (en_US, zh_CN,
zh_HK, zh_TW)
5. Added D-Bus service watcher to wait for notification service
availability
6. Improved notification content with detailed login information and
password warnings
7. Added action button to directly open password change page in control
center

Log: Added configurable login reminder feature with password expiration
warnings

Influence:
1. Test login reminder functionality with DConfig enabled and disabled
states
2. Verify password expiration notifications work independently
3. Check translation loading for different locales
4. Test notification action button opens control center password page
5. Verify autostart behavior after system login
6. Test D-Bus service dependency handling

feat: 增强登录提醒功能并支持配置设置

1. 添加 DConfig 支持登录提醒配置,默认值为关闭
2. 实现密码过期检查作为登录提醒禁用时的备用功能
3. 将 systemd 服务替换为桌面自启动文件以实现更好的会话集成
4. 添加多语言翻译支持(英文、简体中文、繁体中文香港、繁体中文台湾)
5. 添加 D-Bus 服务监视器以等待通知服务可用
6. 改进通知内容,包含详细的登录信息和密码警告
7. 添加操作按钮可直接打开控制中心的密码修改页面

Log: 新增可配置的登录提醒功能,包含密码过期警告

Influence:
1. 测试 DConfig 启用和禁用状态下的登录提醒功能
2. 验证密码过期通知是否独立工作
3. 检查不同语言环境的翻译加载
4. 测试通知操作按钮是否打开控制中心密码页面
5. 验证系统登录后的自启动行为
6. 测试 D-Bus 服务依赖处理
@deepin-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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

@BLumia BLumia merged commit 8110d51 into linuxdeepin:master Dec 4, 2025
13 of 17 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