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

[iOS] 여정 리스트 Scene UI #98

Closed
wants to merge 14 commits into from

Conversation

SwiftyJunnos
Copy link
Member

@SwiftyJunnos SwiftyJunnos commented Nov 23, 2023

❗ 배경

작업 배경에 대한 설명을 작성합니다.
Issue에 대한 링크를 첨부합니다.

close #75

🔧 작업 내역

작업한 내용들을 나열합니다.
간결하게 리스트 업하고, 자세한 설명은 아래 리뷰 노트에서 합니다.

  • 여정 리스트 Scene UI 구현

🧪 테스트 방법

동작을 테스트할 수 있는 방법을 설명합니다.
앱 실행 방법일 수 있고, 유닛 테스트 실행 방법일 수 있습니다.

📝 리뷰 노트

작업 내역에 대한 자세한 설명을 작성합니다.

임의로 ViewModel을 만들어서 Mock 데이터를 넣어뒀습니다.
모델 타입도 임의로 JourneySpot을 만들어뒀습니다.
이후에 연결하며 실제 로직 구현할 때 실제 타입으로 대체하면 될 것 같습니다.

현재 "지난 여정"과 여정 개수를 출력하는 타이틀 부분이 왼쪽에 딱 붙어있는데,
BaseViewController에서 설정한 additionalSafeAreaInset에 맞춰 작업해서 그렇습니다.
머지되고 적용하면 정상적으로 출력될 듯 합니다.

📸 스크린샷

작업한 내용에 대한 스크린샷, 영상 등을 첨부합니다.

데모 영상

@SwiftyJunnos SwiftyJunnos added the 🎨 UI 비즈니스 로직이 없는 UI 작업 label Nov 23, 2023
@SwiftyJunnos SwiftyJunnos added this to the 🌠 여정 조회 milestone Nov 23, 2023
@SwiftyJunnos SwiftyJunnos self-assigned this Nov 23, 2023
Comment on lines 1 to 7
#!/bin/sh

LINTPATH='.swiftlint.yml'
declare -a PATHS=("MSCoreKit" "MSFoundation" "MSUIKit" "MusicSpot")
declare -a PATHS=("MSCoreKit" "MSFoundation" "MSUIKit" "MusicSpot" "Features")
failures=""

for path in "${PATHS[@]}"; do
Copy link
Member

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.

안녕하세요 승현님 😊
commit 할 때마다 Lint 돌리려고 Path 지정해준겁니다... ㅎㅎ

Copy link
Member

Choose a reason for hiding this comment

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

아 githooks precommit으로 사용중인 건가보군요? 감사합니다 ㅎㅎ

Copy link
Member Author

Choose a reason for hiding this comment

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

precommit은 여러가지 이유로 못쓰고 있고..
commit 실행 파일 실행해서 사용하고 있습니다 😅

@SwiftyJunnos SwiftyJunnos linked an issue Nov 25, 2023 that may be closed by this pull request
2 tasks
@SwiftyJunnos SwiftyJunnos changed the title [iOS] 여정 리스트 Scene [iOS] 여정 리스트 Scene UI Nov 27, 2023
@SwiftyJunnos SwiftyJunnos deleted the iOS/story/JourneyListScene branch November 27, 2023 04:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨 UI 비즈니스 로직이 없는 UI 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HomeJourneyList Scene UI
2 participants