Skip to content

πŸ”€ :: (#114) κ²Œμ‹œκΈ€ μ’‹μ•„μš” api#115

Open
glay415 wants to merge 5 commits intomainfrom
114-feed-like-api
Open

πŸ”€ :: (#114) κ²Œμ‹œκΈ€ μ’‹μ•„μš” api#115
glay415 wants to merge 5 commits intomainfrom
114-feed-like-api

Conversation

@glay415
Copy link
Copy Markdown
Member

@glay415 glay415 commented Jun 20, 2022

No description provided.

@glay415 glay415 added the κΈ°λŠ₯ μΆ”κ°€ μƒˆλ‘œμš΄ κΈ°λŠ₯ μš”μ²­μ„ λ‚˜νƒ€λƒ…λ‹ˆλ‹€. label Jun 20, 2022
@glay415 glay415 self-assigned this Jun 20, 2022
@glay415 glay415 linked an issue Jun 20, 2022 that may be closed by this pull request
@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!Β  Β  Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

0.0% 0.0% Coverage
0.0% 0.0% Duplication

PublicFeed feed = publicFeedRepository.findById(feedId)
.orElseThrow(() -> FeedNotFoundException.EXCEPTION);

feed.like();
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

μˆœμ„œλ₯Ό FeedLike 객체λ₯Ό μƒμ„±ν•œ 후에 ν˜ΈμΆœν•˜λŠ” 게 더 쒋을 것 κ°™μ•„μš”

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

γ…‡γ…‹

FeedLike feedLike = FeedLike.builder()
.id(FeedLikeId.builder()
.user(
userFacade.queryCurrentUser()
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

currentUserλŠ” λ”°λ‘œ λ³€μˆ˜λ‘œ λΉΌλŠ” 게 μ–΄λ•Œμš”

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

μ—¬μ„œλ§Œ μ“°λŠ”λ° ꡳ이?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

μ—μž‰

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

κΈ°λŠ₯ μΆ”κ°€ μƒˆλ‘œμš΄ κΈ°λŠ₯ μš”μ²­μ„ λ‚˜νƒ€λƒ…λ‹ˆλ‹€.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

κ²Œμ‹œκΈ€ μ’‹μ•„μš” api

3 participants