Skip to content
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

SettingsRepository 및 닉네임 변경 구현 #151

Merged
merged 4 commits into from
Dec 2, 2022

Conversation

LEEYOONJONG
Copy link
Member

12/1 #127 #128 #145 #146

작업한 내용

닉네임 변경을 위한 흐름을 완성했습니다.

  • SettingsVC, SettingsViewModel, ChangeNicknameUseCase, SettingsRepository에 이르는 전체적인 흐름을 구축했습니다.
  • 서버 명세에 따라 ChangeNickNameEndpoint를 생성했습니다.
  • HTTPMethod 열거형에 PATCH를 추가했습니다.

image

Copy link
Member

@BaeRoNuI BaeRoNuI left a comment

Choose a reason for hiding this comment

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

고생 많으셨습니다!

import Foundation

enum ChangeNicknameEndpoint: Endpoint {
case item(String, String)
Copy link
Member

Choose a reason for hiding this comment

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

사용하다보면 몇번째 parameter이 token인지 혼동이 있을것 같습니다. property label을 붙이면 좋을것 같아요!

Copy link
Member Author

Choose a reason for hiding this comment

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

아 반영하겠씁니다!

Copy link
Collaborator

Choose a reason for hiding this comment

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

저도 이 부분에 동의합니다

Copy link
Collaborator

@rudah7 rudah7 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

@radiantchoi radiantchoi left a comment

Choose a reason for hiding this comment

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

고생 많으셨습니다~ 반응형 패러다임에 대한 이해가 계속 증진되고 계신 것 같습니다.

@radiantchoi radiantchoi merged commit 344967e into develop Dec 2, 2022
@LEEYOONJONG LEEYOONJONG deleted the feature/settings-repository branch December 6, 2022 04:59
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.

4 participants