Skip to content

fix: 과거 notifee 리시버 제거#280

Closed
HeesooJun wants to merge 1 commit intodevelopfrom
comment
Closed

fix: 과거 notifee 리시버 제거#280
HeesooJun wants to merge 1 commit intodevelopfrom
comment

Conversation

@HeesooJun
Copy link
Copy Markdown
Contributor

현재 앱은 @notifee/react-native 9.1.8을 사용합니다
이 버전부터는 부팅/알람 관련 리시버가 라이브러리 app.notifee.core.* 로 제공됩니다.
매니페스트에 남아 있던 app.notifee.receivers.BootReceiver/NotificationReceiver는 이미 존재하지 않는 API라 런타임에 ClassNotFoundException을 일으켜 앱이 부팅 즉시 충돌이 나는 증상을 해결합니다.

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Summary of Changes

Hello @HeesooJun, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 @notifee/react-native 라이브러리 업데이트로 인해 더 이상 유효하지 않은 Android 매니페스트 리시버 선언을 제거하여 앱의 안정성을 향상시키는 것을 목표로 합니다. 이로써 앱 부팅 시 발생하던 ClassNotFoundException으로 인한 충돌을 방지하고, 매니페스트 파일을 최신 라이브러리 사양에 맞게 정리합니다.

Highlights

  • Notifee 리시버 제거: @notifee/react-native 라이브러리 버전 9.1.8 이상에서 더 이상 사용되지 않는 app.notifee.receivers.BootReceiver 및 app.notifee.receivers.NotificationReceiver를 AndroidManifest.xml에서 제거했습니다.
  • 앱 충돌 해결: 이 변경으로 인해 더 이상 존재하지 않는 API 호출로 인한 ClassNotFoundException 발생 및 앱 부팅 시 충돌 문제가 해결됩니다.
  • AndroidManifest.xml 정리: AndroidManifest.xml 파일의 불필요한 리시버 선언을 제거하여 매니페스트를 정리했습니다.
  • 주석 수정: 카카오 리다이렉트 URI 관련 주석의 오타를 수정했습니다.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

이 PR은 AndroidManifest.xml에서 더 이상 사용되지 않는 notifee 리시버를 제거하여 부팅 시 앱 충돌을 해결하는 중요한 수정 사항을 포함하고 있습니다. 핵심적인 변경 사항은 정확하며, 앱 안정성에 꼭 필요한 작업입니다. 다만, 코드 리뷰 과정에서 파일 시작 부분에 불필요한 BOM(Byte Order Mark) 문자가 추가된 점과 주석의 오타 등 몇 가지 사소한 문제를 발견했습니다. 해당 부분에 대한 수정 제안을 코멘트로 남겼으니 확인 후 반영해주시면 코드가 더욱 깔끔해질 것 같습니다.

Comment thread apps/app/android/app/src/main/AndroidManifest.xml
Comment thread apps/app/android/app/src/main/AndroidManifest.xml
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.

1 participant