Skip to content

docs(log-viewer): add Portuguese help manual - #556

Merged
deepin-bot[bot] merged 1 commit into
linuxdeepin:masterfrom
re2zero:docs/log-viewer-manual-pt-br
Oct 27, 2026
Merged

docs(log-viewer): add Portuguese help manual#556
deepin-bot[bot] merged 1 commit into
linuxdeepin:masterfrom
re2zero:docs/log-viewer-manual-pt-br

Conversation

@re2zero

@re2zero re2zero commented Oct 27, 2026 •
edited by sourcery-ai Bot
Loading

Copy link
Copy Markdown
Contributor

Summary

Add the Brazilian Portuguese (pt_BR) translation of the Log Viewer help manual, and normalize the broken en_US desktop manual paths.

Changes

  • Add application/assets/deepin-log-viewer/log-viewer/pt_BR/ (p_log-viewer.md + 8 referenced screenshots).
  • Replace .../en_US/d_log-viewer.md with the delivered version: it removes Windows absolute paths (C:/Users/pifuw/...) and CRLF line endings, and uses the same relative image paths as en_US/p_log-viewer.md.

No other language directory or common asset is modified.

Notes

  • The en_US d_ change is a path/line-ending fix; its content is equivalent to the existing p_ manual (verified after normalizing line endings and equivalent fig/ paths).
  • Verified that every image reference in the new pt_BR manual resolves against the existing common/ and fig/ assets.

Summary by Sourcery

Add Brazilian Portuguese Log Viewer documentation and make the English manual’s asset references portable.

New Features:

  • Add a Brazilian Portuguese translation of the Log Viewer help manual with its referenced screenshots.

Bug Fixes:

  • Correct the English manual’s image links so they resolve using portable relative paths.

Documentation:

  • Provide localized Brazilian Portuguese user documentation for Log Viewer.

@deepin-ci-robot

deepin-ci-robot commented Oct 27, 2026

Copy link
Copy Markdown

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@sourcery-ai

sourcery-ai Bot commented Oct 27, 2026

Copy link
Copy Markdown

Reviewer's Guide

This documentation-only PR adds the Brazilian Portuguese Log Viewer help manual and its screenshots, while fixing the English desktop manual’s portability by replacing broken absolute image paths with repository-relative references and normalizing line endings.

File-Level Changes

Change Details Files
Adds a complete Brazilian Portuguese help manual with localized instructions, UI labels, headings, and screenshots.
  • Adds the translated manual content.
  • Adds eight locale-specific screenshots referenced by the manual.
  • Uses relative paths to shared common icons and locale screenshots.
 application/assets/deepin-log-viewer/log-viewer/pt_BR/p_log-viewer.md
application/assets/deepin-log-viewer/log-viewer/pt_BR/fig/auto_refresh.png
application/assets/deepin-log-viewer/log-viewer/pt_BR/fig/delete_log.png
application/assets/deepin-log-viewer/log-viewer/pt_BR/fig/export_log.png
application/assets/deepin-log-viewer/log-viewer/pt_BR/fig/filter_log.png
application/assets/deepin-log-viewer/log-viewer/pt_BR/fig/refresh_log.png
application/assets/deepin-log-viewer/log-viewer/pt_BR/fig/search_log.png
application/assets/deepin-log-viewer/log-viewer/pt_BR/fig/show_in_fileManager.png
application/assets/deepin-log-viewer/log-viewer/pt_BR/fig/syslog.png
Normalizes the English desktop manual so all image references resolve within the repository.
  • Replaces Windows-specific absolute paths with relative common and figure paths.
  • Normalizes line endings without changing the manual’s substantive content.
  • Aligns the desktop manual’s asset references with the English portable manual.
 application/assets/deepin-log-viewer/log-viewer/en_US/d_log-viewer.md
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

@re2zero
re2zero force-pushed the docs/log-viewer-manual-pt-br branch from dab7b81 to a81cc97 Compare October 27, 2026 08:37
@re2zero
docs(log-viewer): add Portuguese help manual and normalize en_US manu…
205d88e 
…al paths

1. Add the Brazilian Portuguese (pt_BR) manual for Log Viewer with referenced screenshots.
2. Replace Windows absolute paths with relative paths in the en_US desktop manual.
3. Normalize CRLF line endings to LF in the en_US desktop manual.

Log: Add the pt_BR help manual and fix the en_US desktop manual image paths.

Influence:
1. Verify the new pt_BR manual is installed under /usr/share/deepin-manual/manual-assets/application/deepin-log-viewer/log-viewer/pt_BR.
2. Check that all image references in the new pt_BR manual resolve from the installed directory.
3. Verify all en_US manual image references resolve after path normalization.
4. Confirm the normalized en_US content matches the en_US PC manual.
5. Confirm no other language directory is modified.

docs(log-viewer): 新增葡萄牙语帮助手册并修正 en_US 手册路径

1. 添加日志收集工具的巴西葡萄牙语（pt_BR）帮助手册及引用的截图。
2. 将 en_US 桌面版手册中的 Windows 绝对路径替换为相对路径。
3. 将 en_US 桌面版手册的 CRLF 换行统一为 LF。

Log: 新增 pt_BR 帮助手册并修正 en_US 桌面版手册的图片路径。

Influence:
1. 验证新手册安装到 /usr/share/deepin-manual/manual-assets/application/deepin-log-viewer/log-viewer/pt_BR。
2. 检查新 pt_BR 手册中所有图片引用在安装目录下可正常解析。
3. 验证 en_US 手册的所有图片引用在路径修正后可解析。
4. 确认规范化后的 en_US 内容与 en_US PC 版手册一致。
5. 确认没有修改其他语言目录。

PMS：Task-396575   Multica: V-6729
@re2zero
re2zero force-pushed the docs/log-viewer-manual-pt-br branch from a81cc97 to 205d88e Compare October 27, 2026 09:26
@re2zero
re2zero marked this pull request as ready for review October 27, 2026 10:59
@sourcery-ai

sourcery-ai Bot commented Oct 27, 2026

Copy link
Copy Markdown

Sorry @re2zero, you've used your own review budget of 250,000 diff characters for the last 7 days.

You can request another review in 21 hours and 13 minutes by commenting @sourcery-ai review. Upgrade to get a review now.

@deepin-ci-robot

deepin-ci-robot commented Oct 27, 2026

Copy link
Copy Markdown

deepin pr auto review

🤖 AI 代码审查报告

总体评分： 100 分 (通过阈值: 70分)

Pass

📊 总体评价

项目 结果
审查结论 代码审查通过
评分详情 本次提交为纯文档变更，包含两项内容：1) 修复英文帮助手册中的绝对路径为相对路径，消除了泄露本地用户路径的问题；2) 新增葡萄牙语帮助手册翻译。Markdown 语法正确，文档结构清晰，无安全漏洞，无性能影响。

🔍 详细分析

1. 语法逻辑 ✅

评价： 语法正确，逻辑清晰 ✅ 通过

潜在问题：
✅ 未发现明显问题

建议： []

2. 代码质量 ✅

评价： 代码结构清晰，注释完整 ✅ 通过

潜在问题：
✅ 未发现明显问题

建议： []

3. 代码性能 ✅

评价： 性能良好，资源使用合理 ✅ 通过

潜在问题：
✅ 未发现明显问题

建议： []

4. 代码安全 🔒

评价： 存在0个安全漏洞 ✅ 通过

🔐 发现 0 个安全漏洞

安全漏洞详情：
✅ 未发现安全漏洞

建议： []

💡 改进建议代码示例





本报告由 AI 代码审查工具自动生成


      		
    

  





        

            

            

              
            

        

      


      

            
  
  

    
  
    
    

  

  

  


    












      

  
        





      

  
      



  

  @deepin-ci-robot






  



    

      

  

    

      
          
      

      
            deepin-ci-robot
    

      

      

      commented


        Oct 27, 2026



      
    


  


  

    

      

          
    
    

  


        
            
  
      
              Copy link

  

            
  
      
              Copy Markdown

  

        
      

    


    

        



        

    

  





      


        



  
    

      
          
[APPROVALNOTIFIER] This PR is NOT APPROVED


This pull-request has been approved by: lzwind, re2zero


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





      		
    

  





        

            

            

              
            

        

      


      

            
  
  

    
  
    
    

  

  

  


    












      

  
      



  

  @re2zero






  



    

      

  

    

      
          
      

      
            re2zero
    

      

      

      commented


        Oct 27, 2026



      
    


  


  

    

      

          
    
    

  


        
            
  
      
              Copy link

  

            
  
      
              Copy Markdown

  

        
      

    


    

        

  
    Contributor


        

  Author


    

  





      


        



  
    

      
          
/merge


      		
    

  





        

            

            

              
            

        

      


      

            
  
  

    
  
    
    

  

  

  


    












      

  
          
 
  

    
  

  


        
    

        

              @deepin-bot


        

      

              deepin-bot
Bot


            merged commit c0f0c36
          into

          
  
    linuxdeepin:master
  


        Oct 27, 2026

          

            17 checks passed
          

      

    


      

        
  
          
  
    
    


  
      

  







  

  
  

  
    

        

              @re2zero


        

      

        

              re2zero


          
    deleted the
    
      
        docs/log-viewer-manual-pt-br
    
    branch

          October 27, 2026 11:18          
        

      

    





  

  

    
  


  

      


    @re2zero
re2zero



    mentioned this pull request
    
      Oct 27, 2026
    





  


  

      
   Merged

  







  













  
  

    

      

  










  








  
      

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


  










    
        

  

    
    

    

  

    Reviewers
  


    


      

  

    


  
      @lzwind
  
    lzwind



    
      
        
      
    
    lzwind approved these changes





      






    
  
  


  
      

  


      
  

    Assignees
  



        


    No one assigned







  
        


  


  

    Labels
  



    

    None yet







  
        

  

    

        

    Projects
  


        




    None yet




  



  
        


  

    
  

    Milestone
  


      No milestone





    
      
          


  

    

      
        

          
  

    Development
  



            

Successfully merging this pull request may close these issues.





  
  



      
    

  




      

  
  
        

  

    

      3 participants
    

    

        
          @re2zero 
        
          @deepin-ci-robot 
        
          @lzwind