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

[BE] 일정 메모 기능 추가 #941

Merged
merged 6 commits into from
Feb 26, 2024
Merged

[BE] 일정 메모 기능 추가 #941

merged 6 commits into from
Feb 26, 2024

Conversation

pilyang
Copy link
Collaborator

@pilyang pilyang commented Feb 25, 2024

closed #939

PR 내용

일정 도메인에 메모 추가

  • 도메인 엔티티에 필드 추가
    • optional field
    • 최대 100글자
      예시
{
		"title": "4차 데모데이 회의 (아무것도 구현안함)",
		"startDateTime": "2023-07-13 14:00",
		"endDateTime": "2023-07-14 15:59"
		"description": "메모 추가!!" // 친군 없어도 됨 nullable 
}

일정 변경 요청 방법 Update(Patch) 요청시

  • description 필드가 있으면 변경

  • description필드가 null이면 변경 안함 (기존에 메모가 있다면 유지)

  • description필드가 blank ""라면 메모 제거

  • ics배포시 메모도 같이 배포

    • ics 파일 형식상 description:이것은 메모 내용이다추가
    • 엔티티에 필드가 있어야만 생성

Refactor

일정 관련 피드 알림들 모두 제거

  • 제거 이유 : 기존에 사용하지 않고 코드로만 남아있는 레거시.
  • 해당 코드들때문에 코드 복잡성이 증가하고, 다른 기능을 구현하는데 영향을 미침
  • 이전 회의시 일정에대한 알림은 추가하지 않을 생각이라고 하였어서 제거 결정
  • 필요하다면 지금의 도메인 이벤트 방식으로 새롭게 구현하는게 훨씬 깔끔하게 구현 가능

주의사항

  • 배포시 DB 신경슬것!

  • ScheduleNotification table drop

  • schedule table -> description field 추가

Copy link

github-actions bot commented Feb 25, 2024

Test Results

153 files  153 suites   21s ⏱️
500 tests 500 ✅ 0 💤 0 ❌
504 runs  504 ✅ 0 💤 0 ❌

Results for commit 8624655.

♻️ This comment has been updated with latest results.

- description: null 변경하지 않고 유지
- description: blank 메모 제거
refactor
- schedule notification 모두 제거
@pilyang pilyang force-pushed the feat/be/schedule-memo branch from 66e8475 to 2e7ffda Compare February 25, 2024 16:57
@pilyang pilyang merged commit d28ee72 into develop Feb 26, 2024
3 checks passed
@pilyang pilyang deleted the feat/be/schedule-memo branch February 26, 2024 07:44
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.

[BE] 일정도메인에 메모 추가
1 participant