Skip to content

๐Ÿ”€ :: (#105) ๊ฒŒ์‹œ๊ธ€ ์ƒ์„ธ ์ •๋ณด API #124

Open
lyutvs wants to merge 15 commits intomainfrom
105-feed-details
Open

๐Ÿ”€ :: (#105) ๊ฒŒ์‹œ๊ธ€ ์ƒ์„ธ ์ •๋ณด API #124
lyutvs wants to merge 15 commits intomainfrom
105-feed-details

Conversation

@lyutvs
Copy link
Copy Markdown
Collaborator

@lyutvs lyutvs commented Jun 22, 2022

close #105

@lyutvs lyutvs added the ๊ธฐ๋Šฅ ์ถ”๊ฐ€ ์ƒˆ๋กœ์šด ๊ธฐ๋Šฅ ์š”์ฒญ์„ ๋‚˜ํƒ€๋ƒ…๋‹ˆ๋‹ค. label Jun 22, 2022
@lyutvs lyutvs self-assigned this Jun 22, 2022
@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

@AllArgsConstructor
public class ImageUrlListResponse {

private final List<ImageUrlResponse> imageLists;
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.

์ด๊ฑด String ๋ฐฐ์—ด๋กœ ํ•˜๋ฉด ๋  ๊ฒƒ ๊ฐ™์€๋ฐ ์™œ ํˆฌํ”Œ์ž„?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

list ์•„๋‹ˆ์—ˆ์Œ?

Copy link
Copy Markdown
Member

@softpeanut softpeanut Jun 30, 2022

Choose a reason for hiding this comment

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

String ํƒ€์ž…์˜ List

private final CommentAttachmentFileRepository commentAttachmentFileRepository;

@Transactional(readOnly = true)
public FeedDetailsResponse execute(Integer 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.

์•ˆ๋˜๊ฒ ๋‹ค ์ด๊ฑฐ querydsl๋กœ ๋งน๊ธ€์ž
์ค€์„œ๋„ ๋„ˆ๋ฌด ์—ฌ๋Ÿฌ๋ฒˆ ์กฐํšŒํ•˜๋˜๋ฐ

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