Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add create business tag functionallity #408

Merged
merged 6 commits into from
Apr 3, 2025
Merged

feat: add create business tag functionallity #408

merged 6 commits into from
Apr 3, 2025

Conversation

winfredLIN
Copy link
Collaborator

@winfredLIN winfredLIN commented Apr 3, 2025

User description

  1. issue: 新增数据源环境属性和项目所属业务 sqle#3008
  2. 改动内容:增加创建项目的所属业务标签的接口
  3. 接口测试:新增数据源环境属性和项目所属业务 sqle#3008 (comment)

Description

  • 添加业务标签数据模型和数据库迁移

  • 实现业务标签的存储和管理功能

  • 增加API接口和路由以支持业务标签的创建

  • 集成业务标签功能到DMS服务层


Changes walkthrough 📝

Relevant files
Enhancement
7 files
dms_controller.go
实现CreateBusinessTag控制器方法                                                                 
+18/-1   
router.go
新增业务标签的API路由                                                                                         
+1/-0     
business_tag.go
添加业务标签业务逻辑                                                                                             
+54/-0   
business_tag.go
实现业务标签服务方法                                                                                             
+28/-0   
service.go
集成BusinessTagUsecase到DMSService                                                   
+4/-0     
business_tag.go
实现业务标签的存储层                                                                                             
+42/-0   
model.go
定义业务标签数据模型                                                                                             
+6/-0     

Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • - Add BusinessTag struct to the model list for database migration
    - This change enables the system to store and manage business tags
    - Implement BusinessTagUsecase and BusinessTagRepo interfaces
    - Add CreateBusinessTag method to create new business tags
    - Integrate with storage layer to persist business tag data
    - Implement CreateBusinessTag method in DMSController
    - Add corresponding route in APIServer
    - Create BusinessTagUsecase and implement create business tag logic
    - Include error handling and user authentication
    - Initialize and integrate BusinessTagUsecase into DMSService
    - NewBusinessTagUsecase is created and added to the DMSService struct
    Copy link

    github-actions bot commented Apr 3, 2025

    PR Reviewer Guide 🔍

    (Review updated until commit f67cf5b)

    ⏱️ Estimated effort to review: 3 🔵🔵🔵⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No major issues detected

    Copy link

    github-actions bot commented Apr 3, 2025

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    - Changed the length of BusinessName field in BusinessTag struct from varchar(255) to varchar(100)
    Copy link

    github-actions bot commented Apr 3, 2025

    Persistent review updated to latest commit b63f863

    Copy link

    github-actions bot commented Apr 3, 2025

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    - Rename BusinessTagName to Name in BusinessTag struct
    - Update related function parameters and method calls
    - Adjust database model to use Name instead of BusinessName
    Copy link

    github-actions bot commented Apr 3, 2025

    Persistent review updated to latest commit f67cf5b

    Copy link

    github-actions bot commented Apr 3, 2025

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @littleniannian littleniannian merged commit 3478fe0 into main Apr 3, 2025
    1 check passed
    @littleniannian littleniannian deleted the 3008 branch April 3, 2025 09:09
    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