Skip to content

fix: update configuration constants#191

Closed
Kakueeen wants to merge 2 commits intolinuxdeepin:masterfrom
Kakueeen:master
Closed

fix: update configuration constants#191
Kakueeen wants to merge 2 commits intolinuxdeepin:masterfrom
Kakueeen:master

Conversation

@Kakueeen
Copy link
Contributor

  • Renamed control center constants to include versioning for better clarity.
  • Updated the configuration logic to handle both versions of the control center.

Log: fix bug
Bug: https://pms.uniontech.com/zentao/bug-view-301227.html

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Kakueeen

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

@Kakueeen Kakueeen marked this pull request as draft March 20, 2025 05:29
@deepin-bot
Copy link

deepin-bot bot commented Mar 27, 2025

TAG Bot

New tag: 6.0.6
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #194

Kakueeen added 2 commits April 2, 2025 13:18
- Renamed control center constants to include versioning for better clarity.
- Updated the configuration logic to handle both versions of the control center.

Log: fix bug
Bug: https://pms.uniontech.com/zentao/bug-view-301227.html
- Refactored the isActive method to utilize QDBusConnectionInterface for checking service registration.
- Removed outdated logic related to calling allPath and refresh methods, simplifying the availability check.

Log: enhance service availability verification
Bug: https://pms.uniontech.com/bug-view-310679.html
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.

2 participants