Skip to content

Conversation

@jher235
Copy link
Member

@jher235 jher235 commented Sep 27, 2025

Related issue 🛠

Work Description ✏️

  • SoptampUser 의 Android 파트의 파트명 prefix 를 '안드로이드' 에서 '안드'로 사용하도록 수정했습니다.
  • 기존 iOS 를 처리하시는 부분을 하드코딩으로 처리하신 이유가 이전에는 Playground 에서 제공하던 파트명을 현재는 플랫폼에서 제공해주기 때문에 그렇게 하신 것 같은데, 제가 명세서를 확인해보니 내려주는 값들은 비슷해서 하드코딩 보단 Enum이 낫겠다고 생각해서 PlaygroundPart 를 사용하도록 수정했습니다. 그런데 이 플랫폼 측에서 전달받은 데이터를 PlaygroundPart enum 으로 처리한다면 추후 혼란이 있을 수 있을 것 같은데 enum 명을 수정하는게 좋을까요??

++ 스케줄러 cron 표현식 수정

Trouble Shooting ⚽️

Related ScreenShot 📷

Uncompleted Tasks 😅

To Reviewers 📢

@height
Copy link

height bot commented Sep 27, 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.

@jher235 jher235 self-assigned this Sep 28, 2025
@hyerinhwang-sailin
Copy link
Collaborator

hyerinhwang-sailin commented Sep 29, 2025

enum명을 Part나 SoptPart로 수정하는 게 좋을 것 같습니다!
저도 enum이 나을 것 같다고 생각했어서 수정 방향 좋습니다~~
머지 이후에 nickname 반영을 위해 redis 캐시 초기화도 같이 진행해주세요!

그리고 이 pr에 스케쥴러 도는 주기 수정도 같이 부탁드립니다~

- 인증중앙화로 인해 파트 정보를 받아와서 사용하는 곳이 playground에서 platform으로 변경됨. 추후 이런 류의 수정이 발생하더라도 공통적으로 사용할 수 있도록 네이밍을 SoptPart 로 변경
- 기존에는 주석과 다르게 매 시의 4분마다 수행되고 있었기 때문에 이를 수정
@jher235
Copy link
Member Author

jher235 commented Sep 29, 2025

닉네임의 경우 이미 soptampUser 가 생성된 유저의 경우 안드로이드<이름>으로 생성되어 있어서 추후 prod db에 따로 수정 사항을 반영하도록 하겠습니다~

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.

LGTM

@jher235 jher235 merged commit 3edef63 into dev Sep 29, 2025
1 check passed
@jher235 jher235 added the 🛠️ BugFix 버그 해결 label Sep 29, 2025
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.

[BUG] SoptampUser 의 안드로이드 파트명 prefix 수정

2 participants