item_snapshots.edited_by 컬럼을 제거한다 - #1066
Merged
Merged
Conversation
|
Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다. |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yml Review profile: CHILL Plan: Advanced Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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
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.
Situation
edited_by는 1단계(카드 표시값을 포인터 대신 만든 사람 기준으로 계산하고 위시에 item_id 를 되살린다 (1단계) #1054)에서created_by로 복사·흡수됐고, 3단계(edited_by 매핑과 쓰기를 걷어내고 created_by 하나로 둔다 (3단계) #1065)에서 엔티티 매핑과 쓰기까지 사라졌다. 그 상태가 prod 에 promote 됐다.Task
Action
ALTER TABLE item_snapshots DROP COLUMN edited_by.editedByMe응답 필드는 이름만 같고created_by로 판정한다).Result
연관 이슈