-
Notifications
You must be signed in to change notification settings - Fork 1
[Feat] 애스크 대상 상단 고정 유저 리스트 조회 API #863
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
Conversation
| @Getter | ||
| private static final Map<Part, List<Long>> askMembersByPart = Map.of( | ||
| Part.SERVER, List.of(929L,209L), | ||
| Part.IOS, List.of(192L), | ||
| Part.ANDROID, List.of(223L), | ||
| Part.WEB, List.of(945L), | ||
| Part.DESIGN, List.of(930L), | ||
| Part.PLAN, List.of(229L) | ||
| ); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DEV기준 유저 아이디로 되어 있는 것 같은데 혹시 PROD환경에는 코드가 어떻게 올라가나요?
PROD환경의 유저 아이디가 데브에 없는 경우가 많아서 데브 서버에 추후 에러가 날 수도 있을 것 같습니다
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dev 에서 테스트 후, 명단 확정되면 프로덕션 배포할 때 반영할 예정입니다!
| import java.io.Serializable; | ||
|
|
||
| @Getter | ||
| @NoArgsConstructor(access = AccessLevel.PROTECTED) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
해당 코드 변경의 이유가 있을까요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 부분 관련해서, 조회 시에 API 오류가 나서 수정하였습니다!
|
작업하시느라 고생많으셨습니다! |
…und-backend into feat/#862-top-exposure-member
🐬 요약
👻 유형
🍀 작업 내용
🌟 관련 이슈