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

Feature/친구 목록 스크린 로직 개선 #85

Merged
merged 4 commits into from
Aug 31, 2024

Conversation

ottuck
Copy link
Contributor

@ottuck ottuck commented Aug 22, 2024

설명

DB에 있는 친구 데이터(가데이터)를 사용하지 않고 유저 데이터를 가져오도록 로직 변경

변경 사항

자세한 변경사항은 아래 이슈를 참고해주세요.
친구 목록 스크린 개선 #30

체크리스트

< 체크리스트 항목을 확인해 주세요. >

  • 오늘도 행복하게 코딩했는가?
  • 모든 수정 완료 후 make 커맨드를 터미널에서 실행하였는가?
  • release 브랜치를 제대로 최신화 하고 이 브랜치에 merge 했는가?
  • PR 제목은 명확하고 간결한가?
  • PR 에는 하나의 작업에 대한 내용만 포함되었는가?
  • 파일명은 누구나 이해할 수 있게 작성되었는가?
  • camelCase를 사용하였는가? (ThisIsCamelCase, this_is_not_camel_case)
  • 텍스트는 AppStrings.dart 파일에서 가져오고 있는가?
  • 디스코드 채팅방에 완료된 태스크를 알렸는가?

@ottuck ottuck added the enhancement New feature or request label Aug 22, 2024
Copy link
Contributor

@jwson-automation jwson-automation left a comment

Choose a reason for hiding this comment

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

2중 When은 추후에 수정, LGTM!

@jwson-automation jwson-automation merged commit 2da5e49 into release/2024-summer Aug 31, 2024
1 check failed
@jwson-automation jwson-automation deleted the feature/friend-screen branch August 31, 2024 05:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants