Skip to content

Conversation

@huncozyboy
Copy link
Member

@huncozyboy huncozyboy commented Sep 13, 2025

Related issue 🛠

Work Description ✏️

  • UserInternalController 생성
  • POST 유저 생성 internal API 엔드포인트 변경
  • DELETE 유저 삭제 internal API 엔드포인트 변경

To Reviewers 📢

UserInternalController를 새로 생성해주어서 sopt.app.presentation.user에 패키지 경로로 분리해주었는데, 해당 방식이 기존 코드들 컨벤션에 위배되는건 아닌지 의견 주시면 감사하겠습니다 🙇🏻‍♂️

  • 09/15 추가 내용
    Internal API와 관련해서 패키지 분리를 어떻게 가져갈건지
    추후 확장되는 Internal API는 어떤식으로 관리를 해줄건지에 대해서 회의를 완료한 내용을 최종적으로 반영했습니다
image 기존에 존재하던 API들처럼 `package org.sopt.app.presentation.internal;`의 경로로 설정해주었고 지금은 유저 아이디만 반환해주는데 관련된 정보들도 주고 받아야할 필요성이 생기게 될때에 다같이 얘기했었던 내용으로 추가된 `Dto`와 `internal` 관련 서비스단까지 분리해주면 책임 분리 측면에서 더 좋을거같습니다

@huncozyboy huncozyboy self-assigned this Sep 13, 2025
@huncozyboy huncozyboy linked an issue Sep 13, 2025 that may be closed by this pull request
3 tasks
@height
Copy link

height bot commented Sep 13, 2025

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

Copy link
Member

@jher235 jher235 left a comment

Choose a reason for hiding this comment

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

빠른 작업 감사합니다~~

Copy link
Collaborator

@hyerinhwang-sailin hyerinhwang-sailin left a comment

Choose a reason for hiding this comment

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

internal용 controller로 분리해서 더 좋네요~
LGTM

@huncozyboy huncozyboy merged commit 6d55acd into dev Sep 15, 2025
1 check passed
@dlwogus0128
Copy link

잘햇어요

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[MODIFY] 플그 관련 internal API 엔드포인트 수정

4 participants