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

✨ [FEAT] 커뮤니티 메인 API 연결 #461

Merged
merged 16 commits into from
Sep 14, 2022
Merged

✨ [FEAT] 커뮤니티 메인 API 연결 #461

merged 16 commits into from
Sep 14, 2022

Conversation

hwangJi-dev
Copy link
Member

🍎 관련 이슈

closed #445

🍎 변경 사항 및 이유

  • CommunityPostListModel을 PostListResModel로 변경했습니다.

    • 커뮤니티, 자유, 질문(전체에게), 정보, 1:1 질문 게시글 리스트 조회 시 일괄적으로 사용되는 모델이어서 PostListResModel로 통일한 후 CommunityPostListModel 코드를 PostListResModel로 변경해주었습니다.

  • CommunityType Enum을 PostFilterType으로 변경했습니다. API Request시 Post의 Filter type으로 글의 타입이 들어가기 때문에 이를 통일해주었습니다.

🍎 PR Point

  • 커뮤니티 메인 Post List API를 연결하였고, 통신을 성공하여 데이터를 받아온 후 동적으로 뷰에 데이터를 바인딩해주었습니다.

📸 ScreenShot

Simulator.Screen.Recording.-.iPhone.12.-.2022-09-12.at.23.36.38.mp4

@hwangJi-dev
✅ Chore: 게시글 전체 리스트 조회 Response Model 생성 (#445)
5e54d54
@hwangJi-dev
✨ Feat: PostList 조회 API, Service 구현 (#445)
82b173c
@hwangJi-dev
✅ Chore: PostListResModel 수정 및 API 코드 수정 (#445)
c598d09
@hwangJi-dev
✅ Chore: API와의 연관성을 위해 기존 CommunityType을 제거하고 PostFilterType으로 변경 (#445)
0087058
@hwangJi-dev
✅ Chore: 기존 CommunityPostList Model을 PostListResModel로 변경 (#445)
01351fc
@hwangJi-dev
✨ Feat: CommunityMainReactor - 커뮤니티 메인 리스트 API 요청 기능 구현 (#445)
a57fef0
@hwangJi-dev
✨ Feat: 커뮤니티 메인 리스트 API 요청 기능과 뷰 바인딩 (#445)
504dddd
@hwangJi-dev
✅ Chore: 게시글 전체 리스트 조회 Response Model 생성 (#445)
7e5e837
@hwangJi-dev
✨ Feat: PostList 조회 API, Service 구현 (#445)
dbb8fd7
@hwangJi-dev
✅ Chore: PostListResModel 수정 및 API 코드 수정 (#445)
3b988df
@hwangJi-dev
✅ Chore: API와의 연관성을 위해 기존 CommunityType을 제거하고 PostFilterType으로 변경 (#445)
9ebf96f
@hwangJi-dev
✅ Chore: 기존 CommunityPostList Model을 PostListResModel로 변경 (#445)
2ed47d0
@hwangJi-dev
✨ Feat: CommunityMainReactor - 커뮤니티 메인 리스트 API 요청 기능 구현 (#445)
d40e733
@hwangJi-dev
✨ Feat: 커뮤니티 메인 리스트 API 요청 기능과 뷰 바인딩 (#445)
4aa07cc
@hwangJi-dev
✅ Chore: 게시글 리스트 데이터 Model 변경에 따른 코드 수정 (#445)
5adf0e3
@hwangJi-dev
Merge remote-tracking branch 'origin/feature/#445-communityMain-conne…
0e510c5 
…ctAPI' into feature/#445-communityMain-connectAPI
@hwangJi-dev hwangJi-dev added 📡 Network 네트워크 통신 시 사용합니다. 🍎 Feature 새로운 기능 개발시 사용합니다. 💡 Refactor 코드 리펙토링 시 사용합니다. ✨🍕 Jieun 🐰✨ 지은이 연 pr, issue에 사용 🍉 커뮤니티탭 커뮤니티탭 개발시 사용합니다. labels Sep 12, 2022
@hwangJi-dev hwangJi-dev self-assigned this Sep 12, 2022
dev-jungbin
dev-jungbin approved these changes Sep 12, 2022
View reviewed changes
Copy link
Member

@dev-jungbin dev-jungbin left a comment •
edited
Loading

Choose a reason for hiding this comment

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

수고했습니당~!~!

그 이부분 이번 pr에 수정한 부분은 아니라 따로 리뷰는 못 남기지만
커뮤니티부분 seperator Color가 디자인보다 색상이 좀 진한 것 같아요~!!!(코드 보니까 지금 기본인 듯) .gray0으로 나중에 수정해야 할 것 같아염~!

NadoSunbae-iOS/NadoSunbae-iOS/Screen/Community/Reactor/CommunityMainReactor.swift
Comment on lines +107 to +108
// ✅ TODO: Alert Display Protocol화하기
// self.makeAlert(title: "네트워크 오류로 인해\n데이터를 불러올 수 없습니다.\n다시 시도해 주세요.")
Copy link
Member

Choose a reason for hiding this comment

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

옹 이걸 Reactor에서 사용할 수 있게 프로토콜화 하겠다는 거져?~!

Copy link
Member Author

Choose a reason for hiding this comment

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

마자용~~

@hwangJi-dev
Copy link
Member Author

수고했습니당~!~!

그 이부분 이번 pr에 수정한 부분은 아니라 따로 리뷰는 못 남기지만 커뮤니티부분 seperator Color가 디자인보다 색상이 좀 진한 것 같아요~!!!(코드 보니까 지금 기본인 듯) .gray0으로 나중에 수정해야 할 것 같아염~!

대박,,,,,, 정빈 눈썰미 디쟈이너야 모야 고마어 나중에 수정하께~~~

jane1choi
jane1choi approved these changes Sep 14, 2022
View reviewed changes
Copy link
Member

@jane1choi jane1choi left a comment

Choose a reason for hiding this comment

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

확인했습니다!!!

@hwangJi-dev hwangJi-dev merged commit 773eca6 into develop Sep 14, 2022
@hwangJi-dev hwangJi-dev deleted the feature/#445-communityMain-connectAPI branch September 14, 2022 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jane1choi jane1choi jane1choi approved these changes

@dev-jungbin dev-jungbin dev-jungbin approved these changes

Assignees

@hwangJi-dev hwangJi-dev

Labels
🍎 Feature 새로운 기능 개발시 사용합니다. ✨🍕 Jieun 🐰✨ 지은이 연 pr, issue에 사용 🍉 커뮤니티탭 커뮤니티탭 개발시 사용합니다. 📡 Network 네트워크 통신 시 사용합니다. 💡 Refactor 코드 리펙토링 시 사용합니다.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

✨ [FEAT] 커뮤니티 메인 API 연결
3 participants
@hwangJi-dev @dev-jungbin @jane1choi