PR:review api 생성했습니다 by keeprok · Pull Request #69 · PurpleDynamics/PEA_Project

keeprok · 2024-02-28T06:54:14Z

🎯 작업 내용

리뷰 등록
리뷰 목록 조회
리뷰 상세
리뷰 수정
리뷰 삭제

*함수컨벤션
요청종류 / 요청주소 / 정보 / By parameter

✅ 체크 리스트

Main 브랜치 Pull 받기
[x]PR 제목 (컨벤션에 따랐는지) 확인
Label 확인
Assignees 설정 확인
Reviewers 설정 확인

😭 꼭 리뷰 남겨주세요.!!

👍 문제가 없다고 생각된다면...

"FileChanged" Tab 으로 이동합니다.
"Review Changes" Button 클릭합니다.
Comment 입력 (*Optional) 후, "Approve" 선택하고 "Submit Review" 버튼을 클릭합니다.

🙏 코드 수정 요청을 원하신다면...

아래의 템플릿을 복사 하여, 위의 2단계 까지 진행해주세요.

# Request Level
  - [ ] N : "🔥 이대로 Merge 하면 안돼요~!"
  - [ ] M : "🥹 고치면 분명 나아질 게 분명합니다.."
  - [ ] S : "🤷 수정하면 좋지 않을까요?"

# Description

Comment 에 템플릿을 붙혀넣고, where / how / why 3하 원칙(?) 에 따라, 구체적인 요청사항을 작성해주세요.
"Request changes" 선택하고 "Submit Review" 버튼을 클릭합니다.

ijimlnosk

고생하셨습니다 params 수정을 해주셔야 될것같습니다!

ijimlnosk · 2024-02-29T05:28:44Z

src/libs/axios/base/review.js

+	images,
+}) => {
+	const response = await axiosInstance.post(
+		`/review?payList_idx=${payList_idx}`,


주소에 들어가는 params는
params { } , 이렇게 작성하기로 했습니다

fix : 컨벤션 맞춰서 수정했습니다

ijimlnosk · 2024-02-29T05:29:22Z

src/libs/axios/base/review.js

+}
+
+--- 구매 내역 --
+PayList: {


어떤 주석인가요????

response에 대해 적어놓은 것 같은데 없어도 될 것 같아요.

리뷰남겼습니다

jjjiyoung0130

전반적으로 함수명과 주소 부분 수정이 필요할 것 같습니다.
(ByIndex => ByPayListIdx,ByReviewIdx등등)
리뷰 상세의 경우 getReviewByIndex로 작성하였는데 getReviewGetByReviewIdx가 컨벤션에 맞지 않을까요?
진솔님도 언급하신 부분인데 params 부분 수정해주시고 parameter의 스네이크 케이스는 컨벤션에 맞춰서 변경해주세요 :)
response에 대해서도 전부 주석으로 달아주신 거 같은데 그 부분은 없어도 되지않을까 싶네요.

jjjiyoung0130 · 2024-02-29T07:23:59Z

src/libs/axios/base/review.js

+	- 요청결과  200("success") || 400("failure")
+ */
+export const postReviewByIndex = async ({
+	payList_idx,


Request Level

N : "🔥 이대로 Merge 하면 안돼요~!"

M : "🥹 고치면 분명 나아질 게 분명합니다.."

S : "🤷 수정하면 좋지 않을까요?"

Description

snake case는 컨벤션에 맞지 않아요. 수정해주세요 :)

개인적으로 스네이크 케이스는 가져가는게 좋지않을까 합니다....?

fix : 컨벤션 맞춰서 수정했습니다

jjjiyoung0130 · 2024-02-29T07:25:46Z

src/libs/axios/base/review.js

+}
+
+--- 구매 내역 --
+PayList: {


response에 대해 적어놓은 것 같은데 없어도 될 것 같아요.

jjjiyoung0130 · 2024-02-29T14:31:17Z

src/libs/axios/base/review.js

+		{ title, content, ondo, images }
+	);
+
+	return response;


Request Level

N : "🔥 이대로 Merge 하면 안돼요~!"

M : "🥹 고치면 분명 나아질 게 분명합니다.."

S : "🤷 수정하면 좋지 않을까요?"

Description

이 부분만 return값이 response인 이유가 있을까요? 다른 부분은 다 response.data인 것 같은데..

jjjiyoung0130 · 2024-02-29T14:34:21Z

src/libs/axios/base/review.js

+ * @description 리뷰들의 목록을 보여주는 api
+ */
+
+export const getReviewList = async ({ page }) => {


Request Level

N : "🔥 이대로 Merge 하면 안돼요~!"

M : "🥹 고치면 분명 나아질 게 분명합니다.."

S : "🤷 수정하면 좋지 않을까요?"

Description

page가 query string이라면 함수명을 컨벤션에 맞춰 getReviewByPage로 해야하지 않을까요?

동의합니다. 컨벤션이 안지켜진것같네요

jjjiyoung0130 · 2024-02-29T14:34:58Z

src/libs/axios/base/review.js

+/**
+ * 3.리뷰 상세
+ * @function
+ * @parameter review_idx: number - 조회할 리뷰의 idx


Request Level

N : "🔥 이대로 Merge 하면 안돼요~!"

M : "🥹 고치면 분명 나아질 게 분명합니다.."

S : "🤷 수정하면 좋지 않을까요?"

Description

reviewIdx로 수정해야 할 것 같습니다.

jjjiyoung0130 · 2024-02-29T14:36:34Z

src/libs/axios/base/review.js

+		{ params: { review_idx: reviewIdx } },
+		{ title, content, ondo, img_url, main_url, images }
+	);
+	return response;


Request Level

N : "🔥 이대로 Merge 하면 안돼요~!"

M : "🥹 고치면 분명 나아질 게 분명합니다.."

S : "🤷 수정하면 좋지 않을까요?"

Description

이 부분도 return값이 response네요. 하나로 통일시키는게 좋을 것 같습니다.

jjjiyoung0130 · 2024-02-29T14:37:11Z

src/libs/axios/base/review.js

+	const response = await axiosInstance.delete("/review", {
+		params: { review_idx: reviewIdx },
+	});
+	return response;


Request Level

N : "🔥 이대로 Merge 하면 안돼요~!"

M : "🥹 고치면 분명 나아질 게 분명합니다.."

S : "🤷 수정하면 좋지 않을까요?"

Description

이 부분도 return값이 response네요. 하나로 통일시키는게 좋을 것 같습니다.

api요청에따라 데이터 값만 필요한 조회(목록,상세)부분은 response.data로 해주었고 나머지 3요청들은 header 및데이터 외의 값이 필요한게있다고 해서 response로 주었습니다

jjjiyoung0130 · 2024-02-29T14:40:13Z

src/libs/axios/base/review.js

+ * - 요청결과  200("success") || 400("failure")
+ */
+
+export const patchReviewByReviewIdx = async ({ reviewIdx }) => {


Request Level

N : "🔥 이대로 Merge 하면 안돼요~!"

M : "🥹 고치면 분명 나아질 게 분명합니다.."

S : "🤷 수정하면 좋지 않을까요?"

Description

parameter로 reviewIdx만 들어가나요? body부분이 빠진 것 같습니다.

jjjiyoung0130 · 2024-02-29T14:40:39Z

src/libs/axios/base/review.js

+	const response = await axiosInstance.patch(
+		"/review",
+		{ params: { review_idx: reviewIdx } },
+		{ title, content, ondo, img_url, main_url, images }


Request Level

N : "🔥 이대로 Merge 하면 안돼요~!"

M : "🥹 고치면 분명 나아질 게 분명합니다.."

S : "🤷 수정하면 좋지 않을까요?"

Description

snake case가 남아있네요 :)

fix : 컨벤션 수정

hayoung78

PR 올리기전에 한번 더 확인 해주십쇼~~ (저도 그렇긴하지만 ) 뭔가 급하게 올린것같은? 느낌 있습니다요 :) 굳

hayoung78 · 2024-03-01T04:28:28Z

src/libs/axios/base/review.js

+ * @description 리뷰들의 목록을 보여주는 api
+ */
+
+export const getReviewList = async ({ page }) => {


동의합니다. 컨벤션이 안지켜진것같네요

hayoung78 · 2024-03-01T04:30:19Z

src/libs/axios/base/review.js

+ * @description 특정한 하나의 review를 나타내는 API
+ * - 요청한 특정 리뷰가 상세히 조회됩니다


이거 두줄이상이면

@description - 뭐뭐뭐 - 뭐뭐뭐

이런식으로 작성해주세요

리뷰 남겼습니다

hayoung78 · 2024-03-01T04:33:01Z

src/libs/axios/base/review.js

+ 		ex) 전체 온도 70도 / 판매갯수 2 => 35도
+	- 요청결과  200("success") || 400("failure")
+ */
+export const postReviewByPayListIndex = async ({


Request Level

N : "🔥 이대로 Merge 하면 안돼요~!"

M : "🥹 고치면 분명 나아질 게 분명합니다.."

S : "🤷 수정하면 좋지 않을까요?"

Description

Index?

수정부탁바람

fix : 컨벤션 수정

hayoung78 · 2024-03-01T05:11:31Z

src/libs/axios/base/review.js

+ 		ex) 전체 온도 70도 / 판매갯수 2 => 35도
+	- 요청결과  200("success") || 400("failure")
+ */
+export const postReviewByPayListIndex = async ({


수정부탁바람

…into feat/api-review

ijimlnosk

확인했습니다

TransparentDeveloper

오... 굳 진짜 깔끔한데요??

TransparentDeveloper · 2024-03-02T09:19:41Z

src/libs/axios/base/review.js

+	img_url: imgUrl,
+	main_url: mainUrl,
+	images,
+}) => {
+	const response = await axiosInstance.patch(
+		"/review",
+		{ params: { review_idx: reviewIdx } },
+		{ title, content, ondo, img_url: imgUrl, main_url: mainUrl, images }
+	);


Request Level

N : "🔥 이대로 Merge 하면 안돼요~!"

M : "🥹 고치면 분명 나아질 게 분명합니다.."

S : "🤷 수정하면 좋지 않을까요?"

Description

잘 이해가 안돼서 설명을 요청드려요.~

다음과 같이 작성하지 않은 이유가 있을까요??

export const patchReviewByReviewIdx = async ({ ..., imgUrl, // 여기 수정 mainUrl, // 여기 수정 images, }) => { const response = await axiosInstance.patch( "/review", { params: { review_idx: reviewIdx } }, { title, content, ondo, img_url: imgUrl, main_url: mainUrl, images } );

수정했습니다

…into feat/api-review

PR:review api 생성했습니다

df4cccb

keeprok added the feat 추가한 (기능/문제풀이) 간략히 설명 label Feb 28, 2024

keeprok requested review from KR-HeoJU, TransparentDeveloper, hayoung78, ijimlnosk, jjjiyoung0130 and snkchan February 28, 2024 06:54

keeprok self-assigned this Feb 28, 2024

ijimlnosk requested changes Feb 29, 2024

View reviewed changes

jjjiyoung0130 requested changes Feb 29, 2024

View reviewed changes

fix : 컨벤션 맞춰서 수정했습니다

2050637

jjjiyoung0130 requested changes Feb 29, 2024

View reviewed changes

hayoung78 approved these changes Mar 1, 2024

View reviewed changes

hayoung78 self-requested a review March 1, 2024 05:07

hayoung78 requested changes Mar 1, 2024

View reviewed changes

keeprok added 2 commits March 1, 2024 16:03

fix : 컨벤션 수정

7680a20

Merge branch 'main' of https://github.com/PurpleDynamics/PEA_Project …

fcae176

…into feat/api-review

ijimlnosk approved these changes Mar 1, 2024

View reviewed changes

jjjiyoung0130 approved these changes Mar 1, 2024

View reviewed changes

TransparentDeveloper requested changes Mar 2, 2024

View reviewed changes

keeprok added 2 commits March 10, 2024 01:11

Merge branch 'main' of https://github.com/PurpleDynamics/PEA_Project …

05c7536

…into feat/api-review

fix : 컨벤션 수정

c904b2c

		* @description 특정한 하나의 review를 나타내는 API
		* - 요청한 특정 리뷰가 상세히 조회됩니다

Conversation

keeprok commented Feb 28, 2024

🎯 작업 내용

✅ 체크 리스트

😭 꼭 리뷰 남겨주세요.!!

👍 문제가 없다고 생각된다면...

🙏 코드 수정 요청을 원하신다면...

Uh oh!

ijimlnosk left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jjjiyoung0130 left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Request Level

Description

Uh oh!

Choose a reason for hiding this comment

Uh oh!

keeprok Feb 29, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Request Level

Description

Uh oh!

Choose a reason for hiding this comment

Request Level

Description

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Request Level

Description

Uh oh!

Choose a reason for hiding this comment

Request Level

Description

Uh oh!

Choose a reason for hiding this comment

Request Level

Description

Uh oh!

keeprok Mar 1, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Request Level

Description

Uh oh!

Choose a reason for hiding this comment

Request Level

Description

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hayoung78 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jjjiyoung0130 left a comment •

edited

Loading

keeprok Feb 29, 2024 •

edited

Loading

keeprok Mar 1, 2024 •

edited

Loading

keeprok Mar 3, 2024 •

edited

Loading