Skip to content

[Fix] #124 - QA 2차 반영#126

Merged
EunHee-Jeong merged 8 commits intoTeamRecorDream:developfrom
EunHee-Jeong:fix/#124
Jan 19, 2023
Merged

[Fix] #124 - QA 2차 반영#126
EunHee-Jeong merged 8 commits intoTeamRecorDream:developfrom
EunHee-Jeong:fix/#124

Conversation

@EunHee-Jeong
Copy link
Copy Markdown
Member

@EunHee-Jeong EunHee-Jeong commented Jan 19, 2023

👻 작업한 내용

  • 검색, 마이페이지 뷰를 열었다 닫아도 보관함에서 탭바가 사라지지 않도록 수정하였습니다.
  • 레이아웃 마진 값을 다시 조절하였습니다.
  • 같은 감정 아이콘을 다시 클릭했을 때 비활성화 되지 않도록 수정하였습니다.
  • 꿈 기록을 삭제했을 때, 보관함 뷰에서도 함께 fetch 되도록 수정하였습니다. 상세보기 내에서 post 해주는 notification 을 재사용하였습니다.
  • 검색한 키워드를 상세보기로 넘겨주었습니다.

🎤 PR Point

  • 검색한 키워드를 상세보기에서 하이라이트 처리해주는 부분을 겸사겸사 같이 하려고 했는데 우선 제 부분 머지부터 하고 진행하는 것이 좋을 것 같아서 먼저 PR 올립니다!

📮 관련 이슈

@EunHee-Jeong EunHee-Jeong added 으니짱 🍅 담당자 feat 구현·개선 사항에 관련된 내용입니다. fix 버그를 수정합니다. labels Jan 19, 2023
@EunHee-Jeong EunHee-Jeong self-assigned this Jan 19, 2023
L-j-h-c
L-j-h-c previously approved these changes Jan 19, 2023
View reviewed changes
Copy link
Copy Markdown
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.

수고하셨어요~!

...Dream-iOS/Projects/Presentation/Sources/MainTabBarScene/DreamDetail/View/DreamDetailVC.swift Outdated
public var factory: ViewControllerFactory!

private let isModifyDismissed = PublishRelay<Bool>()
let searchedKeyword = PublishRelay<DreamSearchQuery>()
Copy link
Copy Markdown
Contributor

@L-j-h-c L-j-h-c Jan 19, 2023

Choose a reason for hiding this comment

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

어떠한 의도가 있어서 publishRelay를 사용하셨나요?
Reactive한 방식이 필요 없다면 사용을 지양하는 것이 좋아보여요~!

Copy link
Copy Markdown
Member Author

@EunHee-Jeong EunHee-Jeong Jan 19, 2023

Choose a reason for hiding this comment

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

수정하겠습니다아 👍

@EunHee-Jeong EunHee-Jeong merged commit 232412e into TeamRecorDream:develop Jan 19, 2023
@EunHee-Jeong EunHee-Jeong deleted the fix/#124 branch January 19, 2023 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@L-j-h-c L-j-h-c L-j-h-c approved these changes

@Suyeon9911 Suyeon9911 Awaiting requested review from Suyeon9911

Assignees

@EunHee-Jeong EunHee-Jeong

Labels

feat 구현·개선 사항에 관련된 내용입니다. fix 버그를 수정합니다. 으니짱 🍅 담당자

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Fix] QA 2차 반영

2 participants

@EunHee-Jeong @L-j-h-c