Skip to content

feat: improve and integrate label related functions#1290

Merged
benjamin-747 merged 4 commits into
gitmono-dev:mainfrom
yumeowo:main
Jul 31, 2025
Merged

feat: improve and integrate label related functions#1290
benjamin-747 merged 4 commits into
gitmono-dev:mainfrom
yumeowo:main

Conversation

@yumeowo

@yumeowo yumeowo commented Jul 30, 2025

Copy link
Copy Markdown
Contributor

#1148
feat: add New Label
feat: integrate labelList into issue & mr page
feat(UI): complete label selector for issue & mr detail page

@vercel

vercel Bot commented Jul 30, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
mega ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 30, 2025 0:56am

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR integrates label functionality across merge request and issue pages, implementing a complete label management system. The key changes include adding label selectors to issue and MR detail pages, creating a new label dialog, and establishing label-related API hooks.

  • Added label selection functionality to both issue and MR detail pages with proper state management
  • Created a new label creation dialog with color picker and real-time preview
  • Integrated label list management with improved UI and data handling

Reviewed Changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
pages/[org]/mr/[link]/[id].tsx Added label selector integration to MR detail page with useLabelsSelector hook
pages/[org]/labels/index.tsx Refactored labels page with new label dialog, improved search, and better UI
hooks/usePostMRLabels.ts New hook for updating MR labels via API
hooks/usePostLabelNew.ts New hook for creating labels via API
hooks/usePostIssueLabels.ts New hook for updating issue labels via API
hooks/useGetLabelList.ts New hook for fetching label list with caching
components/MrView/index.tsx Integrated label list from API instead of static data
components/MrView/TimelineItems.tsx Added label event handling in timeline
components/Labels/NewLabelDialog.tsx New dialog component for creating labels with color picker
components/Issues/utils/sideEffect.tsx Added useLabelsSelector hook and updated label utilities
components/Issues/IssuesContent.tsx Updated to use dynamic label list from API
components/Issues/IssueDetailPage.tsx Added label selector to issue detail page
Comments suppressed due to low confidence (1)

moon/apps/web/pages/[org]/mr/[link]/[id].tsx:65

  • [nitpick] The variable name 'mrAssignees' should be 'mrAssignees' for consistency, but the original typo 'mrAssgnees' was fixed. However, consider a more descriptive name like 'updateMrAssignees' to better indicate it's a mutation function.
  const { mutate: mrAssignees } = usePostMRAssignees()

Comment thread moon/apps/web/pages/[org]/labels/index.tsx
Comment thread moon/apps/web/pages/[org]/labels/index.tsx
Comment thread moon/apps/web/pages/[org]/labels/index.tsx
Comment thread moon/apps/web/components/Labels/NewLabelDialog.tsx Outdated
Comment thread moon/apps/web/components/Labels/NewLabelDialog.tsx
@benjamin-747
benjamin-747 added this pull request to the merge queue Jul 31, 2025
Merged via the queue into gitmono-dev:main with commit f0a1000 Jul 31, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants