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

[Chore] #187 - 1차 심사 전 출석조회 뷰 레이아웃 조정 #190

Merged
merged 17 commits into from
Apr 19, 2023

Conversation

devxsby
Copy link
Member

@devxsby devxsby commented Apr 19, 2023

🌴 PR 요약

🌱 작업한 브랜치

🌱 PR Point

  • 출석조회 뷰만 조금 다듬었습니다. 코드리뷰 반영하려고했는데 다른 pr로 할게요.

📌 참고 사항

  • 현재 활동기수가 아닌데 플그에서 활동기수로 체크한 경우(슬랙 트러블 슈팅 부분 참고) 출석 조회뷰에서 엠티뷰가 뜨는데 네트워크 얼럿을 추가하려다가 에러 핸들링 부분이 생각보다 까다로워서 따로 하려고 합니다...
    유즈케이스에서 추가하려고 했는데 400에러로 와서 아예 fail로 빠지더라구요 .. 레포지토리에서 처리하기엔 코드가 너무 길어지길래 일단 못했어용 .. .하하.. ㅜ 일단 1차 심사버전이라 활동회원 버그있는 부분은 애초에 못보기 때문에 이정도로만 하고 넘어가는게 좋을거같아요..에러 핸들링 부분만 같이 논의해보면 좋을거같아요 🥲
  • Appearance 일단 Dark로 바꾸어 놓았습니다. 솝탬프만 Light로 하는건 일단 계속 도전해볼게요... 🥲

📸 스크린샷

수료회원으로 보일 때

Simulator Screen Recording - iPhone 14 Pro - 2023-04-19 at 22 46 57

활동회원으로 보일때 (진짜 32기 아닌 경우 → 엠티뷰)

Simulator Screen Recording - iPhone 14 Pro - 2023-04-19 at 22 47 23

활동회원으로 보일때 (진짜 32기 회원인 경우. mock data임)

Simulator Screen Recording - iPhone 14 Pro - 2023-04-19 at 22 50 49

📮 관련 이슈

@devxsby devxsby added Chore 그 외의 잡일/버전 코드 수정, 패키지 구조 변경, 파일 이동, 파일 이름 변경, 레이아웃 조정 등 윤수🥚 labels Apr 19, 2023
@devxsby devxsby self-assigned this Apr 19, 2023
Copy link
Member

@lsj8706 lsj8706 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

@L-j-h-c L-j-h-c left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!!

@devxsby devxsby merged commit 0ebebef into sopt-makers:develop Apr 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Chore 그 외의 잡일/버전 코드 수정, 패키지 구조 변경, 파일 이동, 파일 이름 변경, 레이아웃 조정 등 size/L 윤수🥚
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Chore] 1차 심사 전 뷰 수정
3 participants