-
Notifications
You must be signed in to change notification settings - Fork 2
[FEAT]: 테이블 이미지 조회 응답을 id, url로 묶어서 내려주도록 main에 반영 #147
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.
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
Changes from all commits
4860615
499b92f
b7f43e8
69cff59
bdd3dcf
d7bf7b0
cff7e89
073a21d
20ca6d2
4e815fa
ab2e1ea
a6d71ef
60f6178
d565be2
36f7def
9b777c4
3a40d70
0b5b503
41d4f09
5919a88
1ab6644
df78ea4
48125a9
ad456fd
54d3710
a32cd60
46eb507
a90a8a1
6d541f0
4afc999
4fbe5a8
30ae5a3
e5725e4
7abe68a
4422093
ba066f3
bac7d3c
97f49ef
03153e2
d0a576b
c22438e
8efbbf5
2076181
3601032
ba7fa01
f17b1ed
b179f6b
14435a5
af98092
d3824bd
771651a
2189d8b
3c7685b
3a2acd6
5f75ce1
60ef2cc
11bbf45
460b82c
1a2ef36
e0f0c7c
f8b0d1d
0a73ef8
17b86af
186b979
165b19b
e29373e
d8ad454
181bba0
ee69559
5064cce
7c8fb80
99b5233
3155f47
237c2f9
80cba3f
3edd7a9
529a552
1d0acb7
19e542e
4717113
0c155ad
10b2682
6d06836
dd596c0
5f333b3
ed49ff0
e080e75
1839046
94faf09
ae36c10
a6f9e71
5333ee7
19e6c43
73d45fa
db46b05
b73fe13
405ee87
50fe3bb
3f3cf6a
c63b64e
b2ccaa9
964c100
358a2a6
d406aed
9e65802
0cc15af
a36a456
034d59a
dde9022
68b751a
30128b5
49605db
c50801d
cd5cc8b
4a261c9
988ffd7
aca5023
824aa85
e0f5e63
a3e0385
ab20a8e
0c3ace4
6503e81
fa36abc
8213aae
d785902
4d018d9
d7ef215
d8cd5e1
70ae0d6
19229b0
ba65145
a2eaf70
b3248b4
b16474c
75cceb2
df092c2
a8600d3
9550479
dc981b6
171348c
19a42d2
1062914
3108c2a
9499e14
08eebc6
c9d6a8c
35be75b
3db6f18
732cbb8
1ec6d26
7905e85
e817808
50b2f21
f09b36d
d94dc1d
e57580f
7df4de9
82fb935
7d00784
a78d369
32df4a3
6e18973
64487d9
09ad692
27654a5
19517c2
8be0ee5
04de332
c37c8b0
eb03b8b
5ee8eba
350c98c
9466286
d1865b7
3e99e07
6e01492
635014d
bd3c656
e98b7bf
5945b0d
f395a4d
278f5e7
fd48817
429f91e
9586ebe
84ced95
eb3220e
4e709a7
6f2dbff
74b9d67
c1e7030
15b0d64
dd0329a
cd9d791
7f6d26d
2287aff
1d48fdf
352dc1a
07659dd
e38117a
88af538
f4c8e23
1e9f253
6003fcb
2cd4c13
f87b28b
c31ce87
6c01d63
df7a282
b0c9a6e
15dfdfe
eb7bef7
d3d9d23
c1f5fa3
f544f98
fc27035
5d8ee2f
5aa6b22
05d1c8b
9a2ae15
e68c6fa
7fa07ad
656236b
48d9dd5
6129508
81940b9
3ebd37a
fe84009
33e9284
bbca5db
3f6847c
bdfad23
efd8476
83db327
1609074
6a791aa
86f4806
a2300ee
c586afe
a87cb41
2fc7895
1be6928
e7c5297
5921ec5
080ab7c
0f22d07
553aa65
dddeab0
9bf5737
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -28,6 +28,8 @@ public record StoreSearchDto( | |||||||||
| BigDecimal rating, | ||||||||||
| Integer reviewCount, // 리뷰 도메인이 존재하지 않아 null 허용 | ||||||||||
| double distance, | ||||||||||
| double latitude, | ||||||||||
| double longitude, | ||||||||||
|
Comment on lines
+31
to
+32
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 프리미티브
🛡️ 수정 제안 (nullable 허용 시)- double latitude,
- double longitude,
+ Double latitude,
+ Double longitude,📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||
| String mainImageUrl, | ||||||||||
| boolean isOpenNow | ||||||||||
| ){} | ||||||||||
|
|
||||||||||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -6,21 +6,27 @@ | |||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| public class TableImageResDto { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| @Builder | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| public record UploadTableImageDto( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Long storeId, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| List<String> tableImageUrls | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ){} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| @Builder | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| public record UploadTableImageDto( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Long storeId, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| List<String> tableImageUrls) { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| } | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+9
to
+13
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick | 🔵 Trivial
🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| @Builder | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| public record GetTableImageDto( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Long storeId, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| List<String> tableImageUrls | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ){} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| @Builder | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| public record GetTableImageDto( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Long storeId, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| List<TableImageItem> tableImages) { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| } | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| @Builder | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| public record DeleteTableImageDto( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Long storeId, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| List<Long> deletedTableImageIds | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ){} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| @Builder | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| public record TableImageItem( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Long tableImageId, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| String tableImageUrl) { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| } | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+15
to
+25
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick | 🔵 Trivial
♻️ 선언 순서 정렬 제안+ `@Builder`
+ public record TableImageItem(
+ Long tableImageId,
+ String tableImageUrl) {
+ }
+
`@Builder`
public record GetTableImageDto(
Long storeId,
List<TableImageItem> tableImages) {
}
- `@Builder`
- public record TableImageItem(
- Long tableImageId,
- String tableImageUrl) {
- }📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| @Builder | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| public record DeleteTableImageDto( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Long storeId, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| List<Long> deletedTableImageIds) { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| } | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| } | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧩 Analysis chain
🏁 Script executed:
# 1. StoreConverter.java 파일의 전체 내용 확인 (크기 확인) wc -l src/main/java/com/eatsfine/eatsfine/domain/store/converter/StoreConverter.javaRepository: Eatsfine/BE
Length of output: 137
🏁 Script executed:
Repository: Eatsfine/BE
Length of output: 2649
🏁 Script executed:
Repository: Eatsfine/BE
Length of output: 15923
🏁 Script executed:
Repository: Eatsfine/BE
Length of output: 18123
🏁 Script executed:
Repository: Eatsfine/BE
Length of output: 1859
toSearchDto메소드에서 S3 키를 URL로 변환하지 않고 직접 매핑 — 아키텍처 패턴 불일치toSearchDto는store.getMainImageKey()를 그대로mainImageUrl필드에 매핑하고 있습니다. 반면toDetailDto,toMyStoreDto등 다른 변환 메소드들은 서비스 레이어에서s3Service.toUrl()로 변환된 값을 받아 사용합니다.현재
StoreQueryServiceImpl에서toSearchDto호출 시:mainImageUrl필드명은 URL을 기대하므로, 다른 메소드들과 동일하게 서비스 레이어에서 먼저s3Service.toUrl(store.getMainImageKey())로 변환한 후 전달하거나, 메소드 시그니처를(Store store, Double distance, String mainImageUrl, boolean isOpenNow)로 수정하여 일관성을 유지해야 합니다.🤖 Prompt for AI Agents