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] #207- 출석조회 레이아웃 수정 #208

Merged

Conversation

devxsby
Copy link
Member

@devxsby devxsby commented Apr 24, 2023

🌴 PR 요약

🌱 작업한 브랜치

🌱 PR Point

  • refresh할때 새로운 데이터가 들어올때 레이아웃이 깨지는 오류가 나오지 않도록 조정했어요.
  • info버튼 누르면 회칙이 외부 사파리로 연결되지 않고 웹 뷰로 띄운는거로 바꿨어요.
  • 1차 qa 피드백 받은 상단 오늘의 스케쥴 보이는 부분 위치 → 장소 순으로 보이도록 바꿨어요

📌 참고 사항

  • 출석하기 버튼 부분은 영인님이 작업하신다고 하셔서 안건들였어요.!
    (지금 출석 버튼이 안보이는 때에도 영역이 잡혀서 하단에 검정색 부분으로 조금 잘리네요..!)
  • 무한로딩뷰가 뜨는 경우가 있어서 일단 vc에서 로딩뷰 연결은 해제해놨어요.

📸 스크린샷

Simulator Screen Recording - iPhone 14 Pro - 2023-04-24 at 00 25 57

📮 관련 이슈

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

@0inn 0inn left a comment

Choose a reason for hiding this comment

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

코드 확인했습니다 ~ 고생하셨습니다 . .
로딩뷰는 제가 다시 손보겠습니다 ㅎㅎ
하단 출석하기 버튼은 hidden 처리는 되는데 layout이 업데이트가 안되는거죠 ??
이것도 손보겠습니다 ..^^

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.

수고하셨습니다~~

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.

수고하셨습니다~!!

if let url = URL(string: "https://sopt.org/rules") {
UIApplication.shared.open(url)
}
let safariViewController = SFSafariViewController(url: URL(string: "https://sopt.org/rules")!)
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.

어디에 넣을지 어정쩡해서 일단 그대로 뒀어요. ㅠ
다음에 웹링크만 따로 모아서 정리하는 작업을 해보도록 해요! 🙌🏻

setDefaultLayout()

dateLabel.text = date
placeLabel.text = place
titleLabel.text = I18N.Attendance.today + todaySchedule + I18N.Attendance.dayIs
titleLabel.partFontChange(targetString: todaySchedule,
font: DSKitFontFamily.Suit.bold.font(size: 18))
subtitleLabel.text = description
subtitleLabel.isHidden = ((description?.isEmpty) == nil || description == "")
Copy link
Member

Choose a reason for hiding this comment

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

description?.isEmpty == nil 인 경우는 어떤 경우일까요?
description == nil 이랑 차이가 없을까요?

Copy link
Member Author

Choose a reason for hiding this comment

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

경고 뜨는거 누르고 누르다가 바꼈습니다 호호 . . .

@devxsby devxsby merged commit cc2a0f2 into sopt-makers:develop Apr 24, 2023
@devxsby devxsby deleted the fix/#207-출석조회레이아웃수정 branch April 24, 2023 15:41
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.

[Fix] 출석 조회 레이아웃 수정
4 participants