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

[FIX] 만단위 -> 원단위로 변경 - #136 #137

Merged
merged 1 commit into from
Jul 16, 2024
Merged

Conversation

rlarlgnszx
Copy link
Member

🔥Pull requests

⛳️ 작업한 브랜치

👷 작업한 내용

  • 단위가 만단위에서 원단위로 변경했습니다.
  • 3,5,10,11일때 들어가는 로직으로 변경
    private static void addCostPredicate(List<Predicate> predicates, CriteriaBuilder criteriaBuilder, Root<?> root,
                                         Integer cost) {
        if (cost != null) {
            switch (cost) {
                case 3:
                    predicates.add(criteriaBuilder.lessThanOrEqualTo(root.get("cost"), 30000));
                    break;
                case 5:
                    predicates.add(criteriaBuilder.lessThanOrEqualTo(root.get("cost"), 50000));
                    break;
                case 10:
                    predicates.add(criteriaBuilder.lessThanOrEqualTo(root.get("cost"), 100000));
                    break;
                case 11:
                    predicates.add(criteriaBuilder.greaterThan(root.get("cost"), 100000));
                    break;
                default:
                    break;
            }
        }
    }

🚨 참고 사항

📟 관련 이슈

@rlarlgnszx rlarlgnszx requested review from gardening-y and sjk4618 and removed request for gardening-y July 16, 2024 15:45
@rlarlgnszx rlarlgnszx self-assigned this Jul 16, 2024
Copy link
Contributor

@gardening-y gardening-y left a comment

Choose a reason for hiding this comment

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

수고하셨슈~

@rlarlgnszx rlarlgnszx merged commit f30e31c into develop Jul 16, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] Course가격 만단위 -> 원단위로 수정
2 participants