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] #192 - 코스 상세 페이지 커스텀 메뉴 구현 하였습니다. #195

Conversation

thingineeer
Copy link
Collaborator

@thingineeer thingineeer commented Oct 10, 2023

🌱 작업한 내용

  • 원래 있던 actionSheet 로 올라오는 팝업을 Runnect 만의 커스텀 팝업으로 구현했습니다.
  • DropDown 라이브러리를 설치 하였는데, 라이브러리를 Runnect 앱에 맞게 커스텀을 하였습니다.
    따라서 pod 에 추가할때 pod 'DropDown' 하면 앱 안 돌아갑니다. ‼️

🌱 PR Point

  • 내 코스인지 확인하는 분기는 삼항 연산자로 처리 하였습니다.
  • Action 처리는 따로 함수로 뺏고 로직은 Switch문으로 작성 하였습니다.

📸 스크린샷

구현 내용 내 코스 타 코스
코스 상세 페이지 image 스크린샷 2023-10-11 오후 4 23 23

📮 관련 이슈

@thingineeer thingineeer added Feat 새로운 기능 구현 명진😼 labels Oct 10, 2023
@thingineeer thingineeer self-assigned this Oct 10, 2023
@@ -392,7 +392,8 @@ extension CourseEditVC {
case .success(let result):
let status = result.statusCode
if 200..<300 ~= status {
self.showToast(message: "게시글 수정이 완료되었어요")
self.navigationController?.popViewController(animated: true)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

수정된 화면에서 뒤로 자동으로 돌아가기 위한 코드를 추가했습니다.

@@ -16,6 +16,7 @@ target 'Runnect-iOS' do
pod 'KakaoSDKShare'
pod 'KakaoSDKTemplate'
pod 'FirebaseDynamicLinks'
pod 'DropDown', :git => 'https://github.com/thingineeer/DropDown.git', :commit => '95ee36f7bd925d466033c2c169979f1c574bf3b9'
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

DropDown 라이브러리의 코드를 수정해서 제 로컬에 올리고, 그 코드를 pod로 가져왔습니다.

  • 여기가 문제 부분이었는데 이렇게 따로 가져오지 않으면 변경했던 부분이 저장이 안 돼서 초기화가 될 수 있습니다 ‼️

Copy link
Collaborator

Choose a reason for hiding this comment

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

라이브러리는 어떤 부분들을 수정하신건가요 ?! (Just Question)

Copy link
Collaborator Author

@thingineeer thingineeer Oct 14, 2023

Choose a reason for hiding this comment

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

보통 라이브러리를 수정하지 않고 extension으로 사용하곤 하는데요!

저는 저장 속성을 추가하고 싶어서 라이브러리를 수정하였습니다! (원래는 제약 조건없는 Label 하나가 있었습니다.)
DropDown 파일 안에 보시면 xib 파일이 따로 있는데, 거기에 UIImageView 저장 속성을 추가하였습니다!

( + 아마 라이브러리를 수정하지 않고 하는 방법은 무조건 있을 텐데, 정보가 많이 없었기에 라이브러리를 수정하는 방향으로 진행했습니다)

image

cell.separatorInset = .zero
cell.dropDownImage.image = imageArray[index]
cell.addSubview(lastdividerLineRemove)
}
Copy link
Collaborator Author

@thingineeer thingineeer Oct 13, 2023

Choose a reason for hiding this comment

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

lastDividerLineRemove 뷰를 통해
팝업 뷰에 보이는 맨 아래 선을 지워 주었습니다.

Copy link
Collaborator

@lee-yeonwoo lee-yeonwoo 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
Collaborator

@513sojin 513sojin left a comment

Choose a reason for hiding this comment

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

코드 열심히 읽어봤는데 따로 코멘트 달 부분은 없는 것 같아요!
라이브러리 뜯고 수정하느라 고생하셨습니덩~

@@ -16,6 +16,7 @@ target 'Runnect-iOS' do
pod 'KakaoSDKShare'
pod 'KakaoSDKTemplate'
pod 'FirebaseDynamicLinks'
pod 'DropDown', :git => 'https://github.com/thingineeer/DropDown.git', :commit => '95ee36f7bd925d466033c2c169979f1c574bf3b9'
Copy link
Collaborator

Choose a reason for hiding this comment

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

라이브러리는 어떤 부분들을 수정하신건가요 ?! (Just Question)

@thingineeer thingineeer merged commit dbd86e7 into Runnect:develop Oct 14, 2023
1 check passed
@thingineeer thingineeer deleted the #192---코스-상세-페이지-알림-변경 branch October 14, 2023 08:10
@thingineeer thingineeer changed the title [Feat] #192 - 코스 상세 페이지 커스텀 메뉴 [Feat] #192 - 코스 상세 페이지 커스텀 메뉴 구현 하였습니다. Oct 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feat 새로운 기능 구현 명진😼
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] #192 - 코스 상세 페이지 알림 변경
3 participants