ZinnaChoi / Study-Management

온라인 스터디 관리 시스템: 부재 일정 캘린더, 토론 게시판 및 알림 기능 제공
8 stars 0 forks source link

Task add set notice #90

Closed dayeon-dayeon closed 8 months ago

dayeon-dayeon commented 8 months ago

안녕하세요! 내 정보 페이지 내 알림설정입니다.

image

로그인한 멤버의 알림 설정 상태에 따라 스위치버튼에 표시됩니다.

전체 버튼을 on/ off 시 전체 알림 on/off가 설정되고, 전체 버튼을 제외한 나머지 항목은 개별적으로 설정이 가능합니당

또한 기존에 멤버Id를 가지고 알림 설정 상태를 조회하던 부분을 로그인 정보를 받아오는 것으로 백엔드 수정하였습니다 !

MeMyself-And-I commented 8 months ago

@dayeon-dayeon 다연님 안녕하세요 우선 제 로컬에서 테스트 했을때는 사용자의 notice 정보가 없어서 조회도 불가능하고, 알람 업데이트도 불가능합니다.

확인해보니 회원가입 로직에서 기본값을 설정해야 할 것 같은데 지난 회의동안 회원가입 시에 사용자 알람 설정도 포함되어야 하는건데 누락된건지, 아니면 생각못했던 부분인지 저는 잘 기억이 나지 않습니다 ㅠㅠ 지금으로써는 회원가입 화면에 기본 알람 설정 UI 및 API를 더 추가하고 수정하기에는 시간이 부족해보입니다. 따라서 회원가입 로직(MemberServiceImpl.join() 메소드)에서 Notice 테이블에 기본값 설정(제 생각에는 전부 on으로 설정)하는 JPA 메소드 하나 추가해주실 수 있을까요?

-> 수정 확인했습니다 감사합니다!!! 👍

dayeon-dayeon commented 8 months ago

@MeMyself-And-I 안녕하세요! 피드백 감사합니다!

지난 회의동안 회원가입 시에 사용자 알람 설정도 포함되어야 하는건데 누락된건지, 아니면 생각못했던 부분인지 저는 잘 기억이 나지 않습니다 ㅠㅠ => 저도 기억이 나지 않습니다,,,,,,,ㅠㅠ

따라서 회원가입 로직(MemberServiceImpl.join() 메소드)에서 Notice 테이블에 기본값 설정(제 생각에는 전부 on으로 설정)하는 JPA 메소드 하나 추가해주실 수 있을까요? => 네 회원가입 시 모든 알림 on으로 설정하도록 수정했습니다!