Feature/#67 setting #88

jinukeu · 2025-03-03T14:26:53Z

💡 Issue

Resolved: [Feature] Setting Logic (알림 설정, 회원탈퇴 등) #67

🌱 Key changes

Setting 로직 구현 완료

✅ To Reviewers

FirebaseConsole에서 google-services.json 직접 추가하여야합니다. (일부러 git add 안했어요)
FirebaseConsole는 얍 계정으로 로그인하시면 됩니다.
재원님이 짜신 일부 코드 리팩토링 했습니다. (Api 인터페이스 통합 ...) 관련해서 의견 환영합니다!
201 응답 오면 크래시 나는 문제 해결해놨어용

📸 스크린샷

스크린샷
파일첨부바람

회원가입 시 FCM 토큰과 디바이스 알림 설정 여부를 함께 전송하도록 구현했습니다. Firebase 관련 의존성을 추가하고, 회원가입 요청에 필요한 필드를 추가했습니다.

- 로그인, 회원가입 화면에서 회원가입 완료 후 홈 화면으로 이동하도록 구현 - 알림 설정 API 연동 및 화면 진입 시 설정 상태 반영 - 로그아웃 시 다이얼로그 표시 기능 추가

- 인증된 사용자 API와 레포지토리 구조를 변경했습니다. - 로그인 API를 미인증 사용자 API로 이동했습니다. - 사용자 프로필 API를 인증된 사용자 API로 이동하고, 계정 삭제 기능을 추가했습니다.

로그아웃 시 FCM 토큰 업데이트 로직 추가 및 계정 삭제 UseCase, UI 로직 구현.

coderabbitai · 2025-03-03T14:26:59Z

Important

Review skipped

Auto reviews are limited to specific labels.

🏷️ Labels to auto review (1)

CodeRabbit

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

❤️ Share

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

‼️ IMPORTANT
Auto-reply has been disabled for this repository in the CodeRabbit settings. The CodeRabbit bot will not respond to your replies unless it is explicitly tagged.

Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
- @coderabbitai generate unit testing code for this file.
- @coderabbitai modularize this function.
PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
- @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
- @coderabbitai read src/utils.ts and generate unit testing code.
- @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
- @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

@coderabbitai pause to pause the reviews on a PR.
@coderabbitai resume to resume the paused reviews.
@coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
@coderabbitai full review to do a full review from scratch and review all the files again.
@coderabbitai summary to regenerate the summary of the PR.
@coderabbitai generate docstrings to generate docstrings for this PR.
@coderabbitai resolve resolve all the CodeRabbit review comments.
@coderabbitai configuration to show the current CodeRabbit configuration for the repository.
@coderabbitai help to get help.

Other keywords and placeholders

Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (`.coderabbit.yaml`)

You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
Please see the configuration documentation for more information.
If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

Visit our Documentation for detailed information on how to use CodeRabbit.
Join our Discord Community to get help, request features, and share feedback.
Follow us on X/Twitter for updates and announcements.

ashwon12

고생하셨습니다 !!
fcm 테스트 방법은 따로 없나요 ?!?

ashwon12 · 2025-03-04T06:38:20Z

core/data/src/main/java/com/yapp/core/data/remote/api/UnAuthorizedUserApi.kt

@@ -11,4 +13,9 @@ internal interface UnAuthorizedUserApi {
    suspend fun signUp(
        @Body request: SignUpRequest,
    ): Optional<SignUpResponse>
+
+    @POST("v1/auth/login")
+    suspend fun login(


로그인은 인증 사용자 아닌가요 .. ?!?

앗 아직 인증되지 않은 유저여서 UnAuthroizedUser로 넣었어요 많이 이상하려나요 ... ?

아.. ! 저는 진욱님 코드 봤었을때 UnAuthroizedUser / AuthroizedUser
구분의 기준이 회원가입의 승인을 받았냐 안받았냐로 구분해놓은 줄 알았어요 .. ! 권한이라는 기준으로 구분하시줄알고 ..

권한이 아닌 다른 단어로 바꾸는거 제안 드립니닷 !!
( ex ValidUser / InValidUser )

인증 관련된 Repository, API, UseCase 수정

추가로 리팩토링 진행했습니다! 확실히 재원님 말씀 듣고 다시 보니 오해의 소지가 있었군요
덕분에 네이밍 관련 고민할 수 있는 좋은 기회였습니다 ㅎㅎ

네이밍은 지피티랑 대화하면서 정해봤어요!

+저희 단순히 Repo의 동작을 감싸는 경우에는 UseCase 안쓰고 바로 repo 불러서 쓰는거 어떠세요?

+저희 단순히 Repo의 동작을 감싸는 경우에는 UseCase 안쓰고 바로 repo 불러서 쓰는거 어떠세요?

data-api 모듈 있어서 상관 없을 것 같은데.. 다만 domain 모듈의 필요성이 옅어지는 것 같다는 생각이 듭니닷.. !
추가 커밋은 확인했어요 네이밍 좋아요 !!

jinukeu · 2025-03-04T06:46:59Z

고생하셨습니다 !! fcm 테스트 방법은 따로 없나요 ?!?

아직 Service를 안달아서 테스트 방법은 따로 없습니다 .. ㅜ 그래도 토큰은 잘 찍히는거 보면 괜찮을거같아요

인증 관련 클래스들을 리팩토링하여, 인증 여부에 따라 분리되어있던 API와 Repository를 통합했습니다. 관련 UseCase의 의존성도 수정했습니다.

jinukeu added 7 commits March 3, 2025 17:33

feat/#67: 알림 관련 Repository 및 API 추가

e7e95dd

fix/#67: 201 응답이 내려온 경우 crash 나는 현상

39ffd88

feat/#67: FCM 토큰 및 알림 설정 추가

debafbf

회원가입 시 FCM 토큰과 디바이스 알림 설정 여부를 함께 전송하도록 구현했습니다. Firebase 관련 의존성을 추가하고, 회원가입 요청에 필요한 필드를 추가했습니다.

feat/#67: 설정 화면에 뒤로가기 기능 추가 및 알람 업데이트 기능 추가

17020fe

feat/#67: 계정 삭제 기능 구현

c2e0138

로그아웃 시 FCM 토큰 업데이트 로직 추가 및 계정 삭제 UseCase, UI 로직 구현.

jinukeu added the Feature 기능 추가, 개발 label Mar 3, 2025

jinukeu requested a review from ashwon12 March 3, 2025 14:26

jinukeu self-assigned this Mar 3, 2025

ashwon12 approved these changes Mar 4, 2025

View reviewed changes

refactor/#67: 인증 관련된 Repository, API, UseCase 수정

dd420b0

인증 관련 클래스들을 리팩토링하여, 인증 여부에 따라 분리되어있던 API와 Repository를 통합했습니다. 관련 UseCase의 의존성도 수정했습니다.

jinukeu merged commit 6e2612a into develop Mar 5, 2025
1 check passed

jinukeu deleted the feature/#67-setting branch March 5, 2025 14:05

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Feature/#67 setting #88

Feature/#67 setting #88

Uh oh!

jinukeu commented Mar 3, 2025

Uh oh!

coderabbitai bot commented Mar 3, 2025

Review skipped

Chat

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (`.coderabbit.yaml`)

Documentation and Community

Uh oh!

ashwon12 left a comment

Uh oh!

ashwon12 Mar 4, 2025

Uh oh!

jinukeu Mar 4, 2025

Uh oh!

ashwon12 Mar 4, 2025

Uh oh!

ashwon12 Mar 4, 2025

Uh oh!

jinukeu Mar 4, 2025

Uh oh!

ashwon12 Mar 5, 2025 •

edited

Loading

Uh oh!

jinukeu commented Mar 4, 2025

Uh oh!

Uh oh!

Uh oh!

Feature/#67 setting #88

Feature/#67 setting #88

Uh oh!

Conversation

jinukeu commented Mar 3, 2025

💡 Issue

🌱 Key changes

✅ To Reviewers

📸 스크린샷

Uh oh!

coderabbitai bot commented Mar 3, 2025

Review skipped

Chat

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (.coderabbit.yaml)

Documentation and Community

Uh oh!

ashwon12 left a comment

Choose a reason for hiding this comment

Uh oh!

ashwon12 Mar 4, 2025

Choose a reason for hiding this comment

Uh oh!

jinukeu Mar 4, 2025

Choose a reason for hiding this comment

Uh oh!

ashwon12 Mar 4, 2025

Choose a reason for hiding this comment

Uh oh!

ashwon12 Mar 4, 2025

Choose a reason for hiding this comment

Uh oh!

jinukeu Mar 4, 2025

Choose a reason for hiding this comment

Uh oh!

ashwon12 Mar 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jinukeu commented Mar 4, 2025

Uh oh!

Uh oh!

Uh oh!

CodeRabbit Configuration File (`.coderabbit.yaml`)

ashwon12 Mar 5, 2025 •

edited

Loading