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

공지 API, 공지 상세 API 기능과 Mock 추가 #99

Merged
merged 2 commits into from
Mar 14, 2024
Merged

Conversation

WhiteHyun
Copy link
Member

고민, 과정, 근거 💬

Note

아직 View와 연결하지 않고, 파일만 추가한 상태입니다.

  • NoticeAPINoticeAPISupport를 구현했습니다.
  • 공지 API에서도 페이징을 처리하다보니, Home에서 페이징처리하는 코드에서 공통적인 부분을 뽑아 PagingProvider를 생성했습니다.
  • NoticeViewModel을 추가했습니다. HomeViewModel의 코드에서 NoticeAPI로 변경한 것 빼고는 바뀐 것이 없습니다. 차후 수정할 예정입니다.

PR이 커질 것을 염려하여 미리 올렸습니다 ㅎㅎ..


- Corrected a bug in the Xcode project where the SearchAPI's address path was incorrectly set, ensuring that the project builds and runs correctly without path-related issues.
@WhiteHyun WhiteHyun self-assigned this Mar 13, 2024
@WhiteHyun WhiteHyun requested a review from a team March 13, 2024 10:56
@WhiteHyun WhiteHyun added ⚙️ Settings Settings View 🔨 Implementation New feature or request labels Mar 13, 2024
@WhiteHyun WhiteHyun added this to the v2.0.0 milestone Mar 13, 2024
Copy link
Contributor

@eung7 eung7 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 !

@eung7 eung7 merged commit 484298d into main Mar 14, 2024
5 of 6 checks passed
@eung7 eung7 deleted the feature/settings/97 branch March 14, 2024 02:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 Implementation New feature or request ⚙️ Settings Settings View
Projects
Development

Successfully merging this pull request may close these issues.

세부 공지사항을 서버 API와 연동한다.
2 participants