Skip to content

Conversation

@leebs0521
Copy link
Collaborator

resolved :

📌 과제 설명

위치 기반 조회시 모집중이 아닌 데이터도 조회되는 문제 해결

👩‍💻 요구 사항과 구현 내용

✅ PR 포인트 & 궁금한 점

@leebs0521 leebs0521 added bug Something isn't working 범수 labels Dec 7, 2024
@leebs0521 leebs0521 self-assigned this Dec 7, 2024
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

빌드에 실패했습니다.

@github-actions github-actions bot changed the title [BUG] 위치 기반 조회시 모집중이 아닌 데이터도 조회되는 문제 해결 [BUILD FAIL] [BUG] 위치 기반 조회시 모집중이 아닌 데이터도 조회되는 문제 해결 Dec 7, 2024
@github-actions github-actions bot closed this Dec 7, 2024
BooleanExpression predicate = locationBetween(condition)
.and(keywordEq(condition.keyword()))
.and(statusEq(condition.status()))
.and(isNotDeleted());
Copy link
Collaborator

Choose a reason for hiding this comment

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

최적화 시도 좋은거 같습니다

@Schema(description = "봉사 종료 일시", example = "2024-11-20T12:00:00", type = "string")
@NotNull(message = "봉사 종료 일시는 필수 값입니다.")
LocalDateTime volunteerEndDateTime,
@Future(message = "봉사 종료 일시는 내일부터 가능합니다.")LocalDateTime volunteerEndDateTime,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Future는 어떤 역할의 어노테이션인가요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

날짜 유효성 체크입니다...

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

봉사 모집글 생성과 수정은 당일 기준으로 하루 텀이 있도록(봉사 시작 시간과 종료시간)을 맞춰주도록 했습니다,.

@leebs0521 leebs0521 reopened this Dec 7, 2024
@leebs0521 leebs0521 changed the title [BUILD FAIL] [BUG] 위치 기반 조회시 모집중이 아닌 데이터도 조회되는 문제 해결 [BUG] 위치 기반 조회시 모집중이 아닌 데이터도 조회되는 문제 해결 Dec 7, 2024
Copy link
Collaborator

@ayoung-dev ayoung-dev left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@leebs0521 leebs0521 force-pushed the bug/180-modify-recruit-board-search-by-location branch from d4be1a8 to 3ca169b Compare December 7, 2024 07:35
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 7, 2024

@leebs0521 leebs0521 merged commit 73c6b4d into main Dec 7, 2024
3 checks passed
@leebs0521 leebs0521 deleted the bug/180-modify-recruit-board-search-by-location branch December 7, 2024 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working 범수

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] 위치 기반 조회시 모집중이 아닌 데이터도 조회되는 문제 해결

4 participants