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

[feat] 요청 목록 조회 API 생성 #65

Merged
merged 1 commit into from
Aug 6, 2023

Conversation

psyeon1120
Copy link
Member

🌱 이슈 번호

close #64


💬 기타 사항

  • 지금 파일들이 tab이 8칸씩이나 되어있는데 이 피알처럼 4칸으로 줄이는거 어떤가효? 좋다면 제가 파일 한번 다 정리할게용
  • constant 클래스 이름들은 카멜로 바꿨습니당
  • order 테이블에 리뷰여부 컬럼을 추가했어요! 생각해보니까 상품마다 리뷰하는게 아니라 요청마다 리뷰를 하는거라서 넣었습니다. 근데 쓰다보니까 생각난게 그럼 리뷰 테이블에도 orderId를 넣어야하는 것이 아닌가 라는 생각이 드네요.....

@psyeon1120 psyeon1120 added 🔮 feat 기능을 생성해요 ! ☀️ seoyeon 서연이가 만든 기능이에요 ! 👩‍💻 orders 요청 도메인 기능이에요 ! labels Aug 5, 2023
@psyeon1120 psyeon1120 self-assigned this Aug 5, 2023
Copy link
Collaborator

@chaerlo127 chaerlo127 left a comment

Choose a reason for hiding this comment

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

어엇.. 맞네 orderId가 있어야할거 같네요..
공백 줄이는거 너무 좋아요! 충돌날 수 있어서, 우리 API 다 끝나고 공백 줄이는 건 어떻게 생각하시나욤……?😚😚

const val LOGOUT = "logout"
const val ACTIVE_STATUS = "active"
const val INACTIVE_STATUS = "inactive"
}
}

class USER_EID{
companion object USER_EID{
class UserEID{
Copy link
Collaborator

Choose a reason for hiding this comment

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

고냥 QA로 취직하자 ㅎㅅㅎ

// 요청 목록 조회
@GetMapping
fun getOrderList(@AuthenticationPrincipal userAccount: UserAccount, type: String, status: String): BaseResponse<OrderListRes> {
if (type !in listOf(SELL, ORDER)) return BaseResponse(BaseResponseCode.INVALID_ORDER_TYPE)
Copy link
Member

Choose a reason for hiding this comment

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

옹 이렇게 간편하게 예외처리 할 수 잇군여 !!!!

@@ -0,0 +1,10 @@
package com.psr.psr.order.dto

data class OrderListComp (
Copy link
Member

Choose a reason for hiding this comment

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

comp는 어떤 줄임말인가용

Copy link
Member Author

Choose a reason for hiding this comment

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

component인데 orderRes가 이미 있어서 이렇게 지어봤습니당

Copy link
Member

@sojungpp sojungpp left a comment

Choose a reason for hiding this comment

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

수고하셨숨니다 체고야.

@sojungpp
Copy link
Member

sojungpp commented Aug 6, 2023

🌱 이슈 번호

close #64

💬 기타 사항

  • 지금 파일들이 tab이 8칸씩이나 되어있는데 이 피알처럼 4칸으로 줄이는거 어떤가효? 좋다면 제가 파일 한번 다 정리할게용
  • constant 클래스 이름들은 카멜로 바꿨습니당
  • order 테이블에 리뷰여부 컬럼을 추가했어요! 생각해보니까 상품마다 리뷰하는게 아니라 요청마다 리뷰를 하는거라서 넣었습니다. 근데 쓰다보니까 생각난게 그럼 리뷰 테이블에도 orderId를 넣어야하는 것이 아닌가 라는 생각이 드네요.....

아ㅏ아그러네유,, orderId🥹 파일 탭칸 저도 안 잊게 이슈 만들어두고 마지막에 정리하는거 좋아용

@psyeon1120 psyeon1120 merged commit 244ad95 into develop Aug 6, 2023
1 check passed
@psyeon1120 psyeon1120 deleted the feat/#64-get-order-list branch August 6, 2023 05:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔮 feat 기능을 생성해요 ! 👩‍💻 orders 요청 도메인 기능이에요 ! ☀️ seoyeon 서연이가 만든 기능이에요 !
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] 요청 목록 조회 API
3 participants