-
Notifications
You must be signed in to change notification settings - Fork 0
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: Carpool Register API #19
Conversation
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.
μκ³ λ§μΌμ ¨μ΄μ :)
|
||
@Operation(description = "μΉ΄ν λ±λ‘") | ||
@ApiResponses({ | ||
@io.swagger.v3.oas.annotations.responses.ApiResponse(responseCode = "200", description = "λ±λ‘ μ±κ³΅") |
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.
μ 리μμ€κ° μκΈ΄κ±°λ 201λ‘ κ°λ κ² λ μ’μ κ² κ°μμ!
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.
μμ νμ΅λλ€!
@Column(name = "pickup_location") | ||
private String pickupLocation; |
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.
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.
μ’μ λΆλΆ μ§μ΄μ£Όμ μ κ°μ¬ν©λλ€! κ·Όλ° μ λ κΈμ μ ν μμ κ°μ μμΈμ²λ¦¬λ ν΄λΌμ΄μΈνΈμμ μΆ©λΆν κ²μ¦ν μ μκ³ μμ£Ό λ³λλ μ μλ λΉμ¦λμ€ μꡬμ¬νμ΄κΈ° λλ¬Έμ λλ©μΈμ λ°μμμΌμ λλΈμ²΄νΉνκΈ°λ³΄λ¨ ν΄λΌμ΄μΈνΈμμ κ²μ¦νλκ² μ’λ€κ³ μκ°ν©λλ€!
@NoArgsConstructor(access = PROTECTED) | ||
@AllArgsConstructor(access = PROTECTED) | ||
public class Cost { | ||
private Long money; |
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.
ν¬λ§λΉμ©μ μ΅λ 10μλ€μ!
@EmbeddedId | ||
private CarpoolId carpoolId; | ||
|
||
private String content; |
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.
contentκ° νμΉμμκ² μ ν λ§μ΄κ΅°μ! (μ΅λ 150μ) μλ©μ΄ μ’ λ μ§κ΄μ μ΄λ©΄ μ’μ κ² κ°μλ°..! ν ,, messageλ μ΄λ€κ°μ?
μλ note, memo....
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.
μΉ΄ν κ²μκΈμ λ΄μ©μ΄λΌκ³ μκ°νμ¬ contentλΌκ³ λ€μ΄λ° νμ΅λλ€! λν νμΉ μμ² μλ½ λ νμΉμμκ² λ³΄λ΄λ λ©μμ§λ μμ΄μ ꡬλΆνκΈ° μν΄ μ΄λ κ² νμ΅λλ€!
|
||
@Operation(description = "μΉ΄ν λ±λ‘") | ||
@ApiResponses({ | ||
@io.swagger.v3.oas.annotations.responses.ApiResponse(responseCode = "200", description = "λ±λ‘ μ±κ³΅") |
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.
@Valid μ΄λ Έν μ΄μ λ±λ‘νμ κ±Έ 보λ λ±λ‘μ€ν¨ μΌμ΄μ€λ μμ κ² κ°μμ
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.
μ¬μ€ μ λ μ±κ³΅ν μλ΅λ§ Swaggerμ λ±λ‘νλ νΈμΈλ° νΉμν μ€ν¨ μΌμ΄μ€κ° μκ±°λ μΌμ΄μ€ λ³λ‘ λΆλ₯ν νμμ±μ΄ μκΈ°λ©΄ μΆνμ μΆκ°νλλ‘ νκ² μ΅λλ€!!
@Target({ElementType.PARAMETER, ElementType.TYPE}) | ||
@Retention(RetentionPolicy.RUNTIME) | ||
@AuthenticationPrincipal(expression = "#this == 'anonymousUser' ? null : user") | ||
public @interface CurrentMember { | ||
public @interface CurrentUser { | ||
} | ||
} |
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.
πππππ
public @interface CurrentMember { | ||
public @interface CurrentUser { | ||
} | ||
} |
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.
CurrentMember μμ λ°λ‘ λ CurrentUserλ₯Ό μμ±ν μ΄μ κ° μμΌμ€κΉμ??
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.
μμ νμ΅λλ€!
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.
κ³ μνμ ¨μ΅λλ€ :))
π Summary
Carpool Register API
π©βπ» Contents
Carpool Register API
π Review Note
DTO λ³ν μ± μμ λν κ³ λ―Ό
π£ Related Issue