Skip to content

Feat/week xml 07#14

Open
hyoeunjoo wants to merge 17 commits into
develop-xmlfrom
feat/week-xml-07
Open

Feat/week xml 07#14
hyoeunjoo wants to merge 17 commits into
develop-xmlfrom
feat/week-xml-07

Conversation

@hyoeunjoo
Copy link
Copy Markdown
Collaborator

@hyoeunjoo hyoeunjoo commented May 31, 2024

📌𝘐𝘴𝘴𝘶𝘦𝘴

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

  • Repository를 이용해서 UI와 Data로직 분리했습니다.
  • week 06에서 구현했어서 달라진 부분은 많이 없습니다.

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

화면은 이전과 동일하게 작동됩니다.

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

6주차 코리를 제대로 반영하지 못했습니다..! 시험 끝나고 얼른 반영하겠습니다
부족한 부분들은 언제든지 말씀해주세요!

@hyoeunjoo hyoeunjoo self-assigned this May 31, 2024
Copy link
Copy Markdown
Member

@OliviaYJH OliviaYJH left a comment

Choose a reason for hiding this comment

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

수고하셨슴다!!

): Call<ResponseSignUpDto>
): Response<Unit>

@POST("member/login")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

요기만 다른 서버 통신과 응답 데이터 형식이 다르네요! 혹시 이유가 있을까요?

import com.sopt.now.data.AuthRepoImpl
import com.sopt.now.data.api.ServicePool

class LoginViewModelFactory : ViewModelProvider.Factory {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

멋져요!!

Copy link
Copy Markdown
Member

@chanubc chanubc left a comment

Choose a reason for hiding this comment

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

일단 저번주차와 동일해보여서
저번주차가 머지하고 다시 리뷰해드릴게요!!

한 학기동안 정말 고생많았어요!
고마워요ㅠㅠ

Comment on lines +6 to +9
interface AuthRepository {
suspend fun logIn(authData: AuthEntity): Result<Response<Unit>>
suspend fun signUp(authData: AuthEntity): Result<Response<Unit>>
} No newline at end of file
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

웨케 잘해요?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feat/week7] : 필수과제

3 participants