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

[Fix] 최종 qa 반영 #389

Merged
merged 5 commits into from
Sep 20, 2024
Merged

[Fix] 최종 qa 반영 #389

merged 5 commits into from
Sep 20, 2024

Conversation

hooni0918
Copy link
Member

🔗 연결된 이슈

📄 작업 내용

  • 마이페이지 alert를 띄웟을때 하단 탭바 흐리게 수정
  • 탈퇴하기 이후 화면 백버튼 안생기게 수정
  • 사진 권한 가져올때 물어보기
  • 기타 리젝등 대응
구현 내용 IPhone 15 pro
GIF

👀 기타 더 이야기해볼 점

알람 클릭시 내 모임으로 이동하는 건 아직 구현을 못햇습니다. fcm에 대한 이해가 우선되어야 할것같아서 공부중이에요
appdelegate / Scenedelegate에 있는 코드는 그냥 섞여있는 코드이고 구현중인 코드이니 무시하고 리뷰해주세요

@hooni0918 hooni0918 added 🛠️ fix 버그나 오류 해결시 사용 🧡 JiHoon 쌈뽕한 플러팅 꿀팁을 듣고 싶다면 labels Sep 20, 2024
@hooni0918 hooni0918 self-assigned this Sep 20, 2024
@hooni0918 hooni0918 linked an issue Sep 20, 2024 that may be closed by this pull request
1 task
@hooni0918 hooni0918 changed the title Fix/#383 remove back [Fix] 최종 qa 반영 Sep 20, 2024
Copy link
Member

@youz2me youz2me left a comment

Choose a reason for hiding this comment

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

굿!

Copy link
Contributor

@JinUng41 JinUng41 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다.
앱델이나 신델의 경우, 구현 중인 코드라고 말씀해 주셨는데, 최종 QA 전에 더 반영되어야 하는 내용이 있는지 또는 앞으로 더 수정되어야 할 부분이 있는지 알려주시면 좋을 것 같습니다.

@@ -62,6 +64,13 @@ class AppDelegate: UIResponder, UIApplicationDelegate {
_ application: UIApplication,
didDiscardSceneSessions sceneSessions: Set<UISceneSession>
) {}

func application(_ application: UIApplication, didReceiveRemoteNotification userInfo: [AnyHashable : Any], fetchCompletionHandler completionHandler: @escaping (UIBackgroundFetchResult) -> Void) {
Copy link
Contributor

Choose a reason for hiding this comment

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

이 부분은 개행하면 어떨까용?

Copy link
Member Author

Choose a reason for hiding this comment

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

appdelegate / Scenedelegate에 있는 코드는 그냥 섞여있는 코드이고 구현중인 코드이니 무시하고 리뷰해주세요

Comment on lines +138 to +140
func userNotificationCenter(_ center: UNUserNotificationCenter,
willPresent notification: UNNotification,
withCompletionHandler completionHandler: @escaping (UNNotificationPresentationOptions) -> Void) {
Copy link
Contributor

Choose a reason for hiding this comment

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

개행하니까 너무 읽기 좋은데, 괄호 앞 뒤로 개행했으면 좋겠습니다.
해당 부분을 드래그 후, ctrl + m 눌러보시면 바로 이해하실 것 같습니다.

Copy link
Member Author

Choose a reason for hiding this comment

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

appdelegate / Scenedelegate에 있는 코드는 그냥 섞여있는 코드이고 구현중인 코드이니 무시하고 리뷰해주세요

Comment on lines +150 to +152
func userNotificationCenter(_ center: UNUserNotificationCenter,
didReceive response: UNNotificationResponse,
withCompletionHandler completionHandler: @escaping () -> Void) {
Copy link
Contributor

Choose a reason for hiding this comment

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

이 부분도 괄호 앞 뒤로 개행 부탁드려요.

Copy link
Member Author

Choose a reason for hiding this comment

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

appdelegate / Scenedelegate에 있는 코드는 그냥 섞여있는 코드이고 구현중인 코드이니 무시하고 리뷰해주세요

Comment on lines +16 to 17
var pendingNotificationData: [String: Any]?

Copy link
Contributor

Choose a reason for hiding this comment

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

이 변수는 무엇을 위한 건가요?

Copy link
Member Author

Choose a reason for hiding this comment

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

appdelegate / Scenedelegate에 있는 코드는 그냥 섞여있는 코드이고 구현중인 코드이니 무시하고 리뷰해주세요

Comment on lines +119 to +122
guard let screen = data["screen"] as? String,
let promiseId = data["promiseId"] as? Int else {
return
}
Copy link
Contributor

Choose a reason for hiding this comment

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

else 개행 부탁드립니다.

Copy link
Member Author

Choose a reason for hiding this comment

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

appdelegate / Scenedelegate에 있는 코드는 그냥 섞여있는 코드이고 구현중인 코드이니 무시하고 리뷰해주세요

Comment on lines +38 to +39
private func setupImagePicker() {
imagePicker.delegate = self
Copy link
Contributor

Choose a reason for hiding this comment

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

delegate는 setupDelegate에서 설정해도 좋을 듯 합니다.

Comment on lines +111 to +115
alert.addAction(UIAlertAction(title: "설정", style: .default) { _ in
if let settingsURL = URL(string: UIApplication.openSettingsURLString) {
UIApplication.shared.open(settingsURL)
}
})
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

Copy link
Member

@mmaybei mmaybei left a comment

Choose a reason for hiding this comment

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

굿! 수고하셨습니당

@hooni0918 hooni0918 merged commit 762cc7b into suyeon Sep 20, 2024
@hooni0918 hooni0918 deleted the fix/#383-removeBack branch September 20, 2024 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠️ fix 버그나 오류 해결시 사용 🧡 JiHoon 쌈뽕한 플러팅 꿀팁을 듣고 싶다면
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fix] 최종QA 반영
4 participants