Skip to content

๐Ÿ”€ :: (#69) ์ž„์‹œ์ €์žฅ ๊ฒŒ์‹œ๊ธ€ ์‚ญ์ œ - #67

Open
lyutvs wants to merge 10 commits into
mainfrom
69-localfeed-delete
Open

๐Ÿ”€ :: (#69) ์ž„์‹œ์ €์žฅ ๊ฒŒ์‹œ๊ธ€ ์‚ญ์ œ#67
lyutvs wants to merge 10 commits into
mainfrom
69-localfeed-delete

Conversation

@lyutvs

@lyutvs lyutvs commented Nov 1, 2022

Copy link
Copy Markdown
Contributor

@lyutvs lyutvs added the ๊ธฐ๋Šฅ ์ถ”๊ฐ€ ์ƒˆ๋กœ์šด ๊ธฐ๋Šฅ ์š”์ฒญ์„ ๋‚˜ํƒ€๋ƒ…๋‹ˆ๋‹ค. label Nov 1, 2022
@lyutvs lyutvs self-assigned this Nov 1, 2022
@lyutvs
lyutvs requested a review from jeongyoon05 as a code owner November 1, 2022 11:04
@sonarqubecloud

sonarqubecloud Bot commented Nov 1, 2022

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 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

private final QueryUserSpi queryUserSpi;

@Override
public void execute(UUID feedId) {

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.

API๋ฅผ ํ˜ธ์ถœํ•œ ์œ ์ €๊ฐ€ ๊ฒŒ์‹œ๊ธ€ ์ž‘์„ฑ์ž์ธ์ง€๋ฅผ ํ™•์ธํ•˜๋Š” ๋กœ์ง์„ ์ถ”๊ฐ€ํ•ด์•ผ ํ•  ๊ฑฐ ๊ฐ™์•„์š”

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.

2 participants