[#26]feature: 카드사별 프로모션 관리 #27
Merged
Conversation
This file contains hidden or 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
1. db 에서는 카드사별 프로모션 데이터를 json으로 관리한다. 2. CardBinPromotion 매핑 시 payProvier 에 따라 해당카드사 프로모션 객체로 변환한다.
Closed
f-lab-lyan
reviewed
Nov 20, 2025
| * - 조회 후 PromotionDataPostProcessor에서 PayProvider에 맞게 재변환 | ||
| */ | ||
| @JdbcTypeCode(SqlTypes.JSON) | ||
| @Column(name = "promotion_data", nullable = false, columnDefinition = "json") |
Collaborator
There was a problem hiding this comment.
json으로 저장한다는건, 안에 있는 내용을 특별히 들여다 볼 필요는 없다는 것과 같은 의미인건가요? 😄
Collaborator
Author
There was a problem hiding this comment.
카드사에서 내려준 프로모션 정보는 json으로 관리하고
그 기반으로 사용자에게 혜택정보 제공을 위한 별도 컬럼으로 분리했습니다(카드사마다 관리파라미터가 다르다고 가정했습니다)
현재로서는 json 정보가 위 시점 이후로는 다루지 않고 있습니다.
사실은.. 저번 멘토링때 카드사별로 카드빈객체를 관리한다는 말씀에 db는 json, 서버에서는 각 객체를 정의해둔 상태입니다.
단순 기록용으로만 남게된다면 json컬럼은 삭제하는게 맞을 거 같습니다!
🤔혹시 결제 요청 시에 프로모션에 맞는 키값(?) 이 필요하나요?
f-lab-lyan
reviewed
Nov 20, 2025
| } | ||
|
|
||
| @Override | ||
| public BasePromotionData convertToEntityAttribute(String dbData) { |
f-lab-lyan
reviewed
Nov 20, 2025
| return objectMapper.readValue(json, clazz); | ||
| } | ||
|
|
||
| private Class<? extends BasePromotionData> getPromotionDataClass(PayProvider payProvider) { |
Collaborator
There was a problem hiding this comment.
음 ... 그냥 아이디어인데, 차라리 이걸 그냥 스트링으로 받아서 알아서 처리하는건? 혹은 그냥 map으로 받는건? 어떤가요?
Collaborator
Author
There was a problem hiding this comment.
네 문자열로 받아서 서비스단에서 처리하는게 좋을 거 같습니다 🤩
엔티티 레이어에서 변환하려니 임시 래퍼클래스꺼지 생성되고..
반드시 엔티티에서 처리하지 않아도 되는 부분인 거 같습니다!
저 지금 엔티티,도메인 강박 지옥에 빠진 ..거 같습니다🥲
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.
#26