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.

Sign up for GitHub

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

Jump to bottom

[Chore] #143 - API 헤더 변경 및 각종 뷰 디테일 변경 #148

Merged
merged 8 commits into from
Jan 20, 2022
Merged

[Chore] #143 - API 헤더 변경 및 각종 뷰 디테일 변경 #148

merged 8 commits into from
Jan 20, 2022

Conversation

L-j-h-c
Copy link
Member

@L-j-h-c L-j-h-c commented Jan 20, 2022

🌴 PR 요약

🌱 작업한 브랜치

🌱 작업한 내용

  • 보관함 empty View 분기처리
  • 코드로 참여 글자수 제한 오류 수정
  • 인증하기 팝업 바깥 터치 시 dismiss 기능 구현
  • 습관방 참여 호스트 여부에 따라 default값 조정
  • 불필요한 프린트문 삭제
  • API 헤더 변경(몇몇 멀티파트 폼)

📮 관련 이슈

@L-j-h-c
[Design] TeamSparker#143 - 보관함 empty 이미지 세팅
7b641f7 
보관함 empty 이미지 세팅
@L-j-h-c
[Fix] TeamSparker#143 - API 헤더 변경
ba9d1cc 
API 헤더 변경
@L-j-h-c
[Feat] TeamSparker#143 - 보관함 emptyView 분기처리
97d9562 
보관함 emptyView 분기처리
@L-j-h-c
[Fix] TeamSparker#143 - 코드로 참여 글자수 제한 오류 수정
282d1f1 
코드로 참여 글자수 제한 오류 수정
@L-j-h-c
[Feat] TeamSparker#143 - 인증하기 팝업 바깥 터치 시 dismiss기능 구현
5dc8c47 
인증하기 팝업 바깥 터치 시 dismiss기능 구현
@L-j-h-c
[Fix] TeamSparker#143 - 습관방 참여 버튼 호스트 여부에 따라 default값 조정
63c7c9d 
습관방 참여 버튼 주인이 아닌 경우를 위해 기본값 히든으로 조정
@L-j-h-c
[Fix] TeamSparker#143 - 불필요한 프린트문 삭제
f8ef767 
불필요한 프린트문 삭제
@L-j-h-c L-j-h-c added Fix 오류 해결, 코드 수정 Chore 그 이외의 잡일/ 버전 코드 수정, 패키지 구조 변경, 파일 이동, 파일이름 변경 🧚‍♀️Devil『쥬노』 뀨의 의지 공식후계자입니다 아몰랑 labels Jan 20, 2022
@L-j-h-c L-j-h-c self-assigned this Jan 20, 2022
hyun99999
hyun99999 approved these changes Jan 20, 2022
View reviewed changes
Copy link
Member

@hyun99999 hyun99999 left a comment

Choose a reason for hiding this comment

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

고생했어여

Spark-iOS/Spark-iOS/Source/ViewControllers/HabitRoom/CompleteAuthVC.swift
Comment on lines +40 to +43
self.dismiss(animated: true) {
// let presentingVC = self.presentingViewController
// presentingVC?.navigationController?.popToRootViewController(animated: false)
}
Copy link
Member

Choose a reason for hiding this comment

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

요거 제가 생각해볼게용

@L-j-h-c
[Merge] TeamSparker#143 - 컨플릭트 해결
e307b9a 
# Conflicts:
#	Spark-iOS/Spark-iOS/Resource/Constants/Header.swift
@L-j-h-c L-j-h-c merged commit 3075cec into TeamSparker:develop Jan 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hyun99999 hyun99999 hyun99999 approved these changes

@yangsubinn yangsubinn Awaiting requested review from yangsubinn

Assignees

@L-j-h-c L-j-h-c

Labels
Chore 그 이외의 잡일/ 버전 코드 수정, 패키지 구조 변경, 파일 이동, 파일이름 변경 🧚‍♀️Devil『쥬노』 뀨의 의지 공식후계자입니다 아몰랑 Fix 오류 해결, 코드 수정
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Fix] 서버 함수 네이밍 및 헤더 변경
2 participants
@L-j-h-c @hyun99999