Skip to content
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

fix : update review image #126

Merged
merged 2 commits into from
Sep 19, 2024
Merged

fix : update review image #126

merged 2 commits into from
Sep 19, 2024

Conversation

hanni66
Copy link
Contributor

@hanni66 hanni66 commented Sep 19, 2024

๐Ÿ“Œ Summary

๋ฆฌ๋ทฐ ์ด๋ฏธ์ง€ ์—…๋ฐ์ดํŠธ์‹œ ์ƒ๊ธฐ๋Š” ์˜ค๋ฅ˜ ์ˆ˜์ • ๋ฐ ํ…Œ์ŠคํŠธ ์ฝ”๋“œ ์ถ”๊ฐ€

@hanni66 hanni66 added the fix๐Ÿš‘ ๋ฒ„๊ทธ ์ˆ˜์ • label Sep 19, 2024
@hanni66 hanni66 added this to the COMMUNITY milestone Sep 19, 2024
@hanni66 hanni66 self-assigned this Sep 19, 2024
@hanni66 hanni66 removed the request for review from sshinylee September 19, 2024 15:37
# Conflicts:
#	src/main/java/jjhhyb/deepvalley/community/service/ReviewImageService.java
#	src/test/java/jjhhyb/deepvalley/community/ReviewServiceTest.java
@hanni66 hanni66 merged commit 994d276 into develop Sep 19, 2024
hanni66 added a commit that referenced this pull request Nov 12, 2024
* feat: Add Suggest API (#119)

* feat: Add default thumbnail to valley (#121)

* feat: Add sorting to valley search (#123)

* Member name ์ถ”๊ฐ€ ๋ฐ ๊ณต๊ฐœ ๋ถ€๋ถ„ ์ˆ˜์ • (#124)

* feat : ReviewServiceTest ์ฝ”๋“œ ์ž‘์„ฑ

* feat : ReviewDetailResponse์— ์œ ์ €์˜ ํ”„๋กœํ•„url ์ถ”๊ฐ€ ๋ฐ Null ์ฒดํฌ

* fix : ReviewDetailResponse ๋ถ€๋ถ„ ์ˆ˜์ •

* fix : ๊ธฐ์กด ๋ฆฌ๋ทฐ ์ด๋ฏธ์ง€์—์„œ ์—…๋ฐ์ดํŠธ๋ฅผ ํ†ตํ•ด ๋ฆฌ๋ทฐ ์ด๋ฏธ์ง€๊ฐ€ ์‚ญ์ œ๋˜๋Š” ๊ฒฝ์šฐ ์ถ”๊ฐ€

* feat : ๋ฆฌ๋ทฐ๋ฅผ ๊ธฐ์ค€์œผ๋กœ ์ถ”์ฒœ ๊ณ„๊ณก api ์ถ”๊ฐ€

* feat : ์ถ”์ฒœ ๊ณ„๊ณก api ํ…Œ์ŠคํŠธ ์ฝ”๋“œ ์ถ”๊ฐ€

* feat : 10๊ฐœ์—์„œ 5๊ฐœ๋กœ ์ˆ˜์ •

* fix : deleteByMember ์ถ”๊ฐ€

* fix : response ๋ถ€๋ถ„ ์ˆ˜์ • (์ „์ฒด Tag ๋ณด๋‚ด๊ธฐ)

* fix : ๊ณ„๊ณก ์‚ฌ์ง„ ์กฐํšŒ ๋ถ€๋ถ„ ์ˆ˜์ •

* fix : ๊ณ„๊ณก ์‚ฌ์ง„ ์กฐํšŒ response ์ˆ˜์ •

* fix : member -> set login date, review -> update memberId, placeId

* feat : place ์— ์—…๋ฐ์ดํŠธ ๋˜๋Š” ๋ถ€๋ถ„ ์ˆ˜์ •

* feat : member name ์ถ”๊ฐ€ ๋ฐ ๊ณต๊ฐœ/๋น„๊ณต๊ฐœ ๋ถ€๋ถ„ ์ˆ˜์ •

* feat : ๋ฆฌ๋ทฐ ๊ณต๊ฐœ ๋ฒ”์œ„์— ๋”ฐ๋ผ ํ•„ํ„ฐ๋ง

* Fix : ๋ฆฌ๋ทฐ ์—…๋ฐ์ดํŠธ ์‚ฌ์ง„ ์—…๋กœ๋“œ ๋ถ€๋ถ„ ๋กœ์ง ์ˆ˜์ • (#125)

* feat : ReviewServiceTest ์ฝ”๋“œ ์ž‘์„ฑ

* feat : ReviewDetailResponse์— ์œ ์ €์˜ ํ”„๋กœํ•„url ์ถ”๊ฐ€ ๋ฐ Null ์ฒดํฌ

* fix : ReviewDetailResponse ๋ถ€๋ถ„ ์ˆ˜์ •

* fix : ๊ธฐ์กด ๋ฆฌ๋ทฐ ์ด๋ฏธ์ง€์—์„œ ์—…๋ฐ์ดํŠธ๋ฅผ ํ†ตํ•ด ๋ฆฌ๋ทฐ ์ด๋ฏธ์ง€๊ฐ€ ์‚ญ์ œ๋˜๋Š” ๊ฒฝ์šฐ ์ถ”๊ฐ€

* feat : ๋ฆฌ๋ทฐ๋ฅผ ๊ธฐ์ค€์œผ๋กœ ์ถ”์ฒœ ๊ณ„๊ณก api ์ถ”๊ฐ€

* feat : ์ถ”์ฒœ ๊ณ„๊ณก api ํ…Œ์ŠคํŠธ ์ฝ”๋“œ ์ถ”๊ฐ€

* feat : 10๊ฐœ์—์„œ 5๊ฐœ๋กœ ์ˆ˜์ •

* fix : deleteByMember ์ถ”๊ฐ€

* fix : response ๋ถ€๋ถ„ ์ˆ˜์ • (์ „์ฒด Tag ๋ณด๋‚ด๊ธฐ)

* fix : ๊ณ„๊ณก ์‚ฌ์ง„ ์กฐํšŒ ๋ถ€๋ถ„ ์ˆ˜์ •

* fix : ๊ณ„๊ณก ์‚ฌ์ง„ ์กฐํšŒ response ์ˆ˜์ •

* fix : member -> set login date, review -> update memberId, placeId

* feat : place ์— ์—…๋ฐ์ดํŠธ ๋˜๋Š” ๋ถ€๋ถ„ ์ˆ˜์ •

* feat : member name ์ถ”๊ฐ€ ๋ฐ ๊ณต๊ฐœ/๋น„๊ณต๊ฐœ ๋ถ€๋ถ„ ์ˆ˜์ •

* feat : ๋ฆฌ๋ทฐ ๊ณต๊ฐœ ๋ฒ”์œ„์— ๋”ฐ๋ผ ํ•„ํ„ฐ๋ง

* fix : ๋ฆฌ๋ทฐ ์ˆ˜์ •ํ•˜๊ธฐ ์‚ฌ์ง„ ์—…๋กœ๋“œ ๋ถ€๋ถ„ ์ˆ˜์ •

* fix : update review image And add testcode (#126)

* Fix/review : import ์ˆ˜์ • (#127)

* fix : update review image And add testcode

* Fix : import ์ˆ˜์ •

* Fix : import ์ˆ˜์ •

* Fix : updateReview

* change : ReviewPostRequest

* fix : update ์ž‘์„ฑ ์‹œ deleteimageurls ์ถ”๊ฐ€ ํ›„ add, delete ๊ตฌ๋ถ„

* fix : imageUrls ํƒ€์ž… ์˜ค๋ฅ˜ ์ˆ˜์ •

* fix : s3์—์„œ ์ด๋ฏธ์ง€ ์‚ญ์ œ

* fix : test์šฉ ์ฝ”๋“œ

* fix : update review

* fix : update review

* fix : updateReviewWithImages

* fix : create review

* fix : update review -> delete image

---------

Co-authored-by: RALLY <berygna@gmail.com>
Co-authored-by: hanni66 <72500673+hanni66@users.noreply.github.com>
Co-authored-by: hanni66 <haeunkim0807@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix๐Ÿš‘ ๋ฒ„๊ทธ ์ˆ˜์ •
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants