Skip to content
This repository has been archived by the owner on Nov 21, 2023. It is now read-only.

#64 feed front 추가 #72

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

#64 feed front 추가 #72

wants to merge 2 commits into from

Conversation

bearics
Copy link
Contributor

@bearics bearics commented Feb 21, 2022

  • feed 전용 front 추가
  • feed 목록용 mock api 임시 추가

image

- 아직 리액셔션은 연동 안됨(feed reaction response도 같이 수정이 필요함)

- feed 전용 front 추가
- feed 목록용 mock api 임시 추가
@bearics bearics self-assigned this Feb 21, 2022
@@ -35,6 +35,7 @@ class FeedController(
@ResponseStatus(HttpStatus.OK)
@GetMapping("/v1/feeds")
fun getFeeds(loginUser: LoginUser): FeedsResponse {
return FeedsResponse.create(feedQueryService.getFeeds(loginUserId = loginUser.id))
// return FeedsResponse.create(feedQueryService.getFeeds(loginUserId = loginUser.id))
Copy link
Contributor Author

Choose a reason for hiding this comment

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

mock response 추가라서 나중에 제거 할게요

*/
@Controller
class ArticleController {
@GetMapping("/my-articles")
Copy link
Contributor Author

Choose a reason for hiding this comment

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

front 쪽도 article, feed 용어 분리했습니다.

- 주석 수정
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants