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

64 impl proposal apis post and get #65

Merged
merged 10 commits into from
Dec 2, 2024

Conversation

Gerbera3090
Copy link
Contributor

요약 *

It closes #64
만든 api: ApiPrp004 및 스키마 일부 수정 (스키마 documenatation 완료)
근데 사실 post 하나만 아직 완성돼서 다른 post 및 get도 추가할 예정입니다.

스크린샷

Pasted Graphic 8 Pasted Graphic 9 ![image](https://github.com/user-attachments/assets/31d1b40a-2d36-464d-985e-87639d0949ae)

이후 Task *

  • 커밋이 추가되면 더 올립니다.
  • 리뷰 후 merge되면 남은 prp 관련 api들을 더 만듭니다.

@Gerbera3090 Gerbera3090 linked an issue Nov 29, 2024 that may be closed by this pull request
4 tasks
@Gerbera3090
Copy link
Contributor Author

api prp 005
Pasted Graphic 2
Pasted Graphic
Pasted Graphic 1
Pasted Graphic 3
Pasted Graphic 4
Authorization
Pasted Graphic 6

submittedAt 을 설정하고 난 후:
Pasted Graphic 7
Pasted Graphic 8

이후 수정
Pasted Graphic 9
Pasted Graphic 10

Prp006
image
Pasted Graphic 12
Pasted Graphic 13

Copy link
Contributor

@wjeongchoi wjeongchoi left a comment

Choose a reason for hiding this comment

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

"ASC" | "DESC" 이것만 설명해주세요~ 나머지는 굿

target: Partial<ProjectProposalT>,
isNullCondition?: Partial<Record<keyof ProjectProposalT, boolean>>,
orderByCondition?: Partial<
Record<keyof ProjectProposalT, "ASC" | "DESC">
Copy link
Contributor

Choose a reason for hiding this comment

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

"ASC" | "DESC" 이건 어디서 사용하는건가요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

오름차순 내림차순 나타냅니당

@Gerbera3090 Gerbera3090 merged commit 233a612 into dev Dec 2, 2024
1 check passed
@Gerbera3090 Gerbera3090 self-assigned this Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Impl proposal apis post and get
2 participants