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

[Feat] 인기 게시물을 메인 뷰에서 볼 수 있어요. #382

Merged
merged 11 commits into from
Aug 15, 2024

Conversation

lsj8706
Copy link
Member

@lsj8706 lsj8706 commented Jul 6, 2024

🌴 PR 요약

  • 메이커스에서의 마지막 피쳐가 될 것 같네요. 🫡

🌱 PR Point

  • 이제 메인 뷰에서 솝트 플그의 인기 게시물(핫게)을 볼 수 있어요.
  • 클릭하면 해당 게시물(웹뷰)로 이동해요.

📌 참고 사항

  • 방문자인 경우 기존처럼 인기 게시물 대신 환영 문구만 노출해요.
  • 서버 컨트롤로 핫게의 노출 여부를 결정할 수 있어요.
  • 지금 돌아보면 프로젝트 전반적으로 아쉬운 코드들이 참 많은데 다음 기수 iOS 챕터원들이 잘 매꿔주실거라 믿습니다...🙏

📸 스크린샷

기능 스크린샷
메인뷰 Simulator Screen Recording - iPhone 15 Pro - 2024-07-06 at 22 19 58

@lsj8706 lsj8706 added Feat 새로운 기능 구현 세진🤝🏻 labels Jul 6, 2024
@lsj8706 lsj8706 requested review from elesahich and ffalswo2 July 6, 2024 13:23
@lsj8706 lsj8706 self-assigned this Jul 6, 2024
Copy link

height bot commented Jul 6, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

Copy link
Contributor

@elesahich elesahich left a comment

Choose a reason for hiding this comment

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

굿~~


func initCell(_ hotBoard: HotBoardModel) {
self.hotBoard = hotBoard
titleLabel.text = hotBoard.title
Copy link
Contributor

Choose a reason for hiding this comment

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

세진아 이거 atttributedText로 주는거 있는데 그거 한번 봐볼래?

@lsj8706 lsj8706 merged commit be9c8d2 into develop Aug 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feat 새로운 기능 구현 size/XXL 세진🤝🏻
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants