-
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
M3-238 앱 버전 api #63
M3-238 앱 버전 api #63
Conversation
📝 테스트 커버리지 리포트
|
Qodana for JVMIt seems all right 👌 No new problems were found according to the checks applied 💡 Qodana analysis was run in the pull request mode: only the changed files were checked Contact Qodana teamContact us at [email protected]
|
@Operation(summary = "앱 버전 post", description = "현재 앱 버전을 등록한다.") | ||
@PostMapping("/version") | ||
public Response<?> postVersion( | ||
@Parameter(description = "버전 저장", required = true) | ||
@RequestBody VersionRequest versionRequest | ||
) { | ||
versionService.postVersion(versionRequest); | ||
return Response.createSuccessWithNoData(); | ||
} |
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.
생각해보니 서버가 최신버전 을 주는게 아니라 클라에서 주는 버전을 최신인지 아닌지 검사하는게 좋을 것 같네요.
서버가 최신 버전을 주면 클라에서는 따로 여러 복잡한 로직을 적용못하고 배포 한시점의 로직만 가지고 판단 할 수 밖에 없을 것 같네요
package com.m3pro.groundflip.domain.dto.version; | ||
|
||
import io.swagger.v3.oas.annotations.media.Schema; | ||
import lombok.AllArgsConstructor; | ||
import lombok.Builder; | ||
import lombok.Data; | ||
import lombok.NoArgsConstructor; | ||
|
||
@Data | ||
@NoArgsConstructor | ||
@AllArgsConstructor | ||
@Builder | ||
@Schema(title = "앱 버전 등록") | ||
public class VersionRequest { | ||
@Schema(description = "앱 버전", example = "1.0.1") | ||
private String version; | ||
|
||
} |
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 class VersionController { | ||
private final VersionService versionService; | ||
|
||
@Operation(summary = "앱 버전 get", 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.
앱 버전 get 말고 "업데이트 여부 확인" 같은 표현으로 바꾸는게 좋을 것 같네요
작업 내용*
고민한 내용*
리뷰 요구사항