Skip to content

fix(be): use groupleaderguard in check module#3399

Merged
RyuRaseul merged 7 commits intomainfrom
t2519-checkservice-authorization
Feb 26, 2026
Merged

fix(be): use groupleaderguard in check module#3399
RyuRaseul merged 7 commits intomainfrom
t2519-checkservice-authorization

Conversation

@lshtar13
Copy link
Contributor

@lshtar13 lshtar13 commented Feb 3, 2026

Description

  1. import groupleaderguard in several resolvers in check module
  2. modify validaterequest logic in check service to consider group
  3. add groupid field in corresponding bru files

Additional context


Before submitting the PR, please make sure you do the following

Closes TAS-2519

1. import groupleaderguard in several resolvers in check module
2. modify validaterequest logic in check service to consider group
3. add groupid field in corresponding bru files
@lshtar13 lshtar13 self-assigned this Feb 3, 2026
Copilot AI review requested due to automatic review settings February 3, 2026 06:24
@lshtar13 lshtar13 added this to Codedang Feb 3, 2026
@github-project-automation github-project-automation bot moved this to Pending ✋ in Codedang Feb 3, 2026
@skkuding-bot
Copy link

skkuding-bot bot commented Feb 3, 2026

Syncing Preview App Succeeded

Application: frontend
Revision: 2b5bdb1e56cb377e3666edd12f5b328d3e2f71fb
Health Status: Healthy

Open Preview | View in Argo CD

This comment was marked as resolved.

@notion-task-integration
Copy link

notion-task-integration bot commented Feb 3, 2026

This comment was marked as resolved.

@skkuding-bot
Copy link

skkuding-bot bot commented Feb 3, 2026

Syncing Preview App Succeeded

Application: frontend
Revision: 87cb2178c1786a9349fee595faba179ec72d4fe6
Health Status: Healthy

Open Preview | View in Argo CD

@skkuding-bot
Copy link

skkuding-bot bot commented Feb 4, 2026

Syncing Preview App Succeeded

Application: frontend
Revision: fd1ad7c4c8154d297289b229a413b202a1042c02
Health Status: Healthy

Open Preview | View in Argo CD

RyuRaseul

This comment was marked as resolved.

@github-project-automation github-project-automation bot moved this from Pending ✋ to In Progress 🏃 in Codedang Feb 26, 2026
@RyuRaseul RyuRaseul self-requested a review February 26, 2026 09:00
@RyuRaseul

This comment has been minimized.

@github-project-automation github-project-automation bot moved this from In Progress 🏃 to Approved 👌 in Codedang Feb 26, 2026
@RyuRaseul
Copy link
Contributor

RyuRaseul commented Feb 26, 2026

PR이 너무 깔끔하네요
컨플릭은 제가 해결할테니 노션이나 PR 설명(노션 추천)에 PR 설명(쿼리 의도 등) 정리해주시면 너무 좋을 것 같습니다!
LGTM!

@RyuRaseul RyuRaseul enabled auto-merge February 26, 2026 11:04
@RyuRaseul RyuRaseul added this pull request to the merge queue Feb 26, 2026
Merged via the queue into main with commit 0f5ac72 Feb 26, 2026
11 checks passed
@RyuRaseul RyuRaseul deleted the t2519-checkservice-authorization branch February 26, 2026 11:11
@github-project-automation github-project-automation bot moved this from Approved 👌 to Done ✔️ in Codedang Feb 26, 2026
@lshtar13
Copy link
Contributor Author

PR이 너무 깔끔하네요 컨플릭은 제가 해결할테니 노션이나 PR 설명(노션 추천)에 PR 설명(쿼리 의도 등) 정리해주시면 너무 좋을 것 같습니다! LGTM!

노션 태스크 페이지에 설명 추가하였습니다! 너무 적게 적은 듯 한데 부족한 부분 있으면 말씀해주세요!

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

Projects

Status: Done ✔️

Development

Successfully merging this pull request may close these issues.

3 participants