Skip to content

chore: Update dman's resources#475

Merged
deepin-bot[bot] merged 2 commits intolinuxdeepin:masterfrom
dengzhongyuan365-dev:master
Jun 19, 2025
Merged

chore: Update dman's resources#475
deepin-bot[bot] merged 2 commits intolinuxdeepin:masterfrom
dengzhongyuan365-dev:master

Conversation

@dengzhongyuan365-dev
Copy link
Contributor

@dengzhongyuan365-dev dengzhongyuan365-dev commented Jun 19, 2025

chore: Update dman's resources

  • Update dman's resources

log: Update dman's resources

Summary by Sourcery

Update and restructure the Device Manager user documentation in English and Chinese locales, including renaming files, reorganizing sections, unifying terminology, refining wording, and updating related SVG assets.

Enhancements:

  • Refactor user guide layout by introducing a 'Getting Started' section, standardizing headings, and reorganizing run/close instructions.
  • Unify terminology and improve consistency across English and Chinese documentation for sections like hardware info, context menus, and driver management operations.

Documentation:

  • Rename and update markdown files for en_US, zh_HK, zh_TW, and zh_CN locales with revised titles, tables, and instructional content.
  • Refine wording throughout the documentation to enhance clarity and align UI action descriptions consistently.

Chores:

  • Update SVG assets: add new icon file, remove deprecated icons, and refresh common icon resources across the documentation.

Copy link

@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.

Hey @dengzhongyuan365-dev - I've reviewed your changes - here's some feedback:

  • Double-check the new icon asset deepin_devicema96.svg and removal of deepin_devicemanager_96.svg to ensure the filenames and references aren’t typos and still resolve correctly.
  • Standardize the markdown filenames and top-level headers across all locales (e.g. avoid embedding the filename in the header and adopt a consistent naming pattern like device-manager_<lang>.md).
  • Ensure section titles, ordering, and terminology (e.g. Driver Management vs Driver management) are consistently capitalized and aligned across the English and all localized files.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Double-check the new icon asset `deepin_devicema96.svg` and removal of `deepin_devicemanager_96.svg` to ensure the filenames and references aren’t typos and still resolve correctly.
- Standardize the markdown filenames and top-level headers across all locales (e.g. avoid embedding the filename in the header and adopt a consistent naming pattern like `device-manager_<lang>.md`).
- Ensure section titles, ordering, and terminology (e.g. Driver Management vs Driver management) are consistently capitalized and aligned across the English and all localized files.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@github-actions
Copy link

TAG Bot

TAG: 6.0.41
EXISTED: no
DISTRIBUTION: unstable

@sourcery-ai
Copy link

sourcery-ai bot commented Jun 19, 2025

Reviewer's Guide

This pull request refactors and harmonizes the Device Manager documentation across all locales, restructuring sections for clarity, standardizing terminology and menu actions, updating hardware tables and driver management instructions, and adjusting related SVG assets.

Flow diagram for right-click device context menu actions (updated)

flowchart TD
    A[User right-clicks device in details area] --> B{Menu options}
    B --> C[Copy info]
    B --> D[Refresh device info]
    B --> E[Export info]
    B --> F[Disable/Enable device]
    B --> G[Update driver]
    B --> H[Uninstall driver]
    B --> I[Allow Wake Computer]
    C --> J[Copy all or partial info]
    D --> K[Reload device info]
    E --> L[Export as txt/docx/xls/html]
    F --> M[Device enabled/disabled]
    G --> N[Select driver folder]
    N --> O[System detects drivers]
    O --> P[User selects and updates]
    P --> Q[Authentication]
    Q --> R[Update success/failure]
    H --> S[Confirm uninstall]
    S --> T[Uninstall success/failure]
    I --> U[Enable/disable wake by device]
Loading

File-Level Changes

Change Details Files
Restructure documentation sections and refine wording
  • Rename and reformat main markdown files (e.g., device-manager_EN.md)
  • Introduce a distinct ‘Getting Started’ section
  • Refine Overview, Running, and Closing subsections for clarity
assets/.../en_US/device-manager.md
assets/.../zh_HK/device-manager.md
assets/.../zh_TW/device-manager.md
assets/.../zh_CN/device-manager.md
Standardize section headings and menu action labels
  • Unify headings like ‘Hardware Information’, ‘Driver Management’, ‘Main Menu’
  • Align menu item names (Theme, Help/说明, About, Exit/结束)
  • Ensure consistent icon-menu callouts across locales
assets/.../en_US/device-manager.md
assets/.../zh_HK/device-manager.md
assets/.../zh_TW/device-manager.md
assets/.../zh_CN/device-manager.md
Update hardware information tables
  • Uniform hardware field names (Processor, Motherboard, etc.)
  • Harmonize description phrasing across all tables
  • Fix typos and capitalization in table headers
assets/.../en_US/device-manager.md
assets/.../zh_HK/device-manager.md
assets/.../zh_TW/device-manager.md
assets/.../zh_CN/device-manager.md
Consolidate right-click menu operation descriptions
  • Merge Copy, Refresh, Export, Disable/Enable entries into uniform format
  • Clarify Update Driver and Uninstall Driver steps
  • Streamline Wake Computer via Mouse/Keyboard/Network sections
assets/.../en_US/device-manager.md
assets/.../zh_HK/device-manager.md
assets/.../zh_TW/device-manager.md
assets/.../zh_CN/device-manager.md
Revamp driver management and online installation instructions
  • Unify driver update detection and installation steps
  • Standardize note and feedback prompts
  • Simplify batch install and result-handling descriptions
assets/.../en_US/device-manager.md
assets/.../zh_HK/device-manager.md
assets/.../zh_TW/device-manager.md
assets/.../zh_CN/device-manager.md
Adjust SVG assets for the Device Manager UI
  • Add new deepin_devicema96.svg asset
  • Remove deprecated deepin_devicemanager_96.svg
assets/.../common/deepin_devicema96.svg
assets/.../common/deepin_devicemanager_96.svg

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: dengzhongyuan365-dev, lzwind

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

@dengzhongyuan365-dev
Copy link
Contributor Author

/forcemerge

@deepin-bot
Copy link
Contributor

deepin-bot bot commented Jun 19, 2025

This pr force merged! (status: unstable)

@deepin-bot deepin-bot bot merged commit 2bf3af7 into linuxdeepin:master Jun 19, 2025
17 of 18 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