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

[Feat/347] 모임 일정 친구 초대 기능 구현 #365

Open
wants to merge 38 commits into
base: dev
Choose a base branch
from

Conversation

nahy-512
Copy link
Member

Related Issue

Type of change

  • Feature : 새로운 기능 추가
  • Bug fix : 버그 수정
  • Refactor : 코드 리팩토링 작업
  • Document : 문서작업
  • Test : 테스트 코드 작성 및 테스트 작업
  • Style : 코드 스타일 및 포맷팅 작업
  • Chore : 패키지 매니저, 라이브러리 업데이트 등의 작업

PR Desciption

변경 사항 설명

  • 모임 일정 참석자를 초대하는 로직을 구현했습니다.
  • 모임 생성 단계에서는 참석자 선택을 하지 않고, 생성 후에 참석자 추가 의사를 묻는 팝업을 띄워줍니다.
Screen_Recording_20250217-063516.1.mp4

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

  • 모임 편집 모드에서 진입 시 초대할 친구/초대된 친구/모든 친구를 구분해줘야 하는데 그 부분에서 조금 애를 먹었습니다.

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

고민 중인 사항

첨부 자료

nahy-512 and others added 30 commits December 10, 2024 08:08
- 생성 시 친구 초대 안 보이게 하기

Related to: #347
- GetFriendsUseCase 활용

Related to: #347
- 초대할 친구 아이템 UI 구현
- 전체 선택 취소, 초대 현황 표시, 초대하기 버튼

Related to: #347
- 초대할 친구 리사이클러뷰 adapter 연결
- 친구 선택/해제에 따른 처리

Related to: #347
- Friend 데이터 클래스의 userId
- ScheduleRepository에서의 getGuestInvitationLink

Related to: #347
- 커뮤니티 알림 화면 통합
- 기획 변경에 따른 모임 알림 화면 삭제

Related to: #347
- 일정-카테고리 선택 다이얼로그에서 편집 화면까지 바로 이동
- 화면 이동 방식 수정

Related to: #346
- 토스트 메시지 표시

Related to: #346
- 서버 명칭과 달라서 생겼던 문제
- isShared -> shared로 이름 변경

Related to: #346
- 카테고리 설정 화면에서는 휴지통 버튼 숨기기
- 일정 화면에서는 생성/편집 모드에 따라 노출 여부 결정하기

Related to: #346
- 변경사항이 있을 경우 경고 팝업 표시

Related to: #347
- 이미 초대한 친구는 삭제가 불가능함
- 정책 변경에 따른 수정 사항

Related to: #347
…e-347

# Conflicts:
#	app/src/main/res/values/strings.xml
- MoimFragment에서 이동한 화면(모임 일정, 친구 초대)에서 result로 데이터를 받아 오던 변수 이름 수정
- initMoimResultData로 초기화 함수 작성

Related to: #347
- isInvitedFriendExist 값 추가
- 검색창 GONE 처리

Related to: #347
@nahy-512 nahy-512 requested a review from edv-Shin February 16, 2025 21:41
@nahy-512 nahy-512 linked an issue Feb 16, 2025 that may be closed by this pull request
10 tasks
@nahy-512 nahy-512 self-assigned this Feb 16, 2025
@nahy-512 nahy-512 added the ✨ feature 기능 구현 관련 label Feb 16, 2025
Comment on lines +74 to 82
<!-- <TextView-->
<!-- android:id="@+id/item_friend_introduction_tv"-->
<!-- android:layout_width="wrap_content"-->
<!-- android:layout_height="wrap_content"-->
<!-- android:layout_marginTop="6dp"-->
<!-- android:text="@{friend.introduction}"-->
<!-- tools:text="친구가 직접 작성한 한 줄 소개"-->
<!-- style="@style/subtext_12" />-->

Copy link
Collaborator

Choose a reason for hiding this comment

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

안쓰는거라면 지워주세요!

@edv-Shin
Copy link
Collaborator

오류대응은 딱히 없어도 무방한가요?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feature 기능 구현 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] 모임 일정 친구 초대 기능 구현
2 participants