#262 [feat] 지원서 있는지 없는지 확인하는 GET API 추가 #263
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
KWY0218
reviewed
Feb 28, 2024
| @@ -13,9 +13,10 @@ | |||
| public interface HairModelApplicationJpaRepository extends JpaRepository<HairModelApplication, Long> { | |||
| Boolean existsByModelId(Long modelId); | |||
|
|
|||
| Optional<HairModelApplication> findByModelId(Long modelId); | |||
| Optional<HairModelApplication> findFirstByModelIdOrderByCreatedAtDesc(Long modelId); | |||
There was a problem hiding this comment.
p3
지원서를 한장만 유지하기 위해서 해당 기능을 넣는 것으로 알고 있습니다.
혹시 order by를 하는 이유가 있을까요?
There was a problem hiding this comment.
@KWY0218 해당 모델ID로 작성했던 지원서가 전체 조회가 되서 List 형태로 값이 불러와질텐데,
그 경우에 모든 지원서에 대해 만료되었는지 체크하기 보다 최근에 만들어진 지원서에 대한 만료 여부만 파악하면 될거 같아서 최신것을 가져오도록 했습니다! 생각해보니 모든 값이 다 정렬되고 가져올텐데 더 좋은 방법이 있을지 생각해봐야겠네요..!
There was a problem hiding this comment.
@hellozo0
아 만료된 지원서가 포함되는 것은 생각하지 못했었네요!
굳 좋습니다!
|
유효한것이 아니라면 2가지로 나눠서 (만료된것, 작성한것이 없음! ) 으로 나눠서 404 리턴하도록 수정했습니다~ 
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
관련 이슈번호
해결하는 데 얼마나 걸렸나요? (예상 작업 시간 / 실제 작업 시간)
해결하려는 문제가 무엇인가요?
**만료되지 않은 지원서!
어떻게 해결했나요?
어떻게 하는 것이 좋을까요....... 아니면 hasApplication을 없애고 nonData 형식으로 유효한게 있으면 200, 아니면 다 유효하지 않음 이런식으로 하는게 좋을까요??? 지금 방법은 뭔가 일을 2번해야하는것 같아서 고민입니다~