전체 공지 사항 - Githubissues

ckdtls1124 commented 1 year ago

프로젝트 관련하여, 공유할 필요가 있는 파일은 master Branch에 올리도록 하겠습니다.

ckdtls1124 commented 1 year ago

각자 맡은 부분의 단계별 프론트 작업이 완료되는 대로, 페이지 캡처본을 각자 Branch에 올려주시고, 댓글 남겨주시면 감사하겠습니다.

댓글 예시 ) 이지창 - 경찰관 상세 조회 페이지 완료

ckdtls1124 commented 1 year ago

프론트 작업 및 백 작업을 할 때, Entity명을 변수명 및 메소드의 앞에 항상 들어가도록 합시다.

예시 ) AttendEntity의 변수명은 모두 Attend로 시작하고, 메소드 및 클래스명도 Attend로 시작

wkdgytjs commented 1 year ago

프론트 작업 하실 때 각자 포인트로 넣을 색상 있으시면 남색은 #003478 노란색은 #ffc400 빨간색은 혹시 몰라서.. #ad0000 통일해주시면 좋을 것 같습니다!

ckdtls1124 commented 1 year ago

이지창 : 회원 CRUD, 허인경 : 사건사고 CRUD, 김득주 : 로그인 & 시큐리티, 장효선 : 게시판 CRUD, 강창신 : 근태 CR, 민원 RU, 결재CRUD, mainIndex

챗봇 - 강창신, 허인경, 장효선 캘린더 - 장효선, 이지창 네이버 API 사용 - 강창신, 김득주

ckdtls1124 commented 1 year ago

로그인 페이지를 제외한 담당자분들 께서는, master branch의 design frame을 참고하시고 View 제작을 해주시면 됩니다.

ckdtls1124 commented 1 year ago

중간 점검은 3월 23일에 하겠습니다.

jichang-lee commented 1 year ago

03/23 Member CRUD , Dept CRUD 및 관련된 HTML&CSS&JS merge push 완료했습니다 merge에 문제 있으신 경우 말씀해주시면 수정하겠습니다

jichang-lee commented 1 year ago

0327 push

왼쪽 메뉴바 fragment 하여 common 디렉토리 안에 추가
role 디렉토리에 권한(ADMIN,MEMBER) 이 있는 enum 파일 추가 현재 회원&부서 서비스 로직은 이상없는데 sec:authorize로 ADMIN 권한이 있어야만 볼 수 있는 보안 태그가 작동하지 않아서 좀 더 확인해야 할 것 같습니다. 혹여나 서비스 테스트가 필요한 분들은 merge-test pull 하셔서 개인 파일 넣고 테스트 돌려보심 되겠습니다. 최대한 빠르게 다시 업데이트 하겠습니다~

jichang-lee commented 1 year ago

0328 11:15 push

어제 문제되었던 sec:authorize 권한 설정이 안된 것 해결 -> UserDetailService 에서 Role을 get하지 않아서 User가 DB에 저장되어도 권한을 갖지않아서 문제가 됨
왼쪽 메뉴바 fragment 했던 것 (leftWrap.html) 안에 link= reset.css 적용
관리자가 사원등록 시 해당 유저에 권한을 선택하여 부여할 수 있음 -> 관리자 -> 'ADMIN' , 사원 -> 'MEMBER' (테스트할 때 편리성을 위한)

각 팀장님 및 팀원님들께서 테스트 하실 때는 첫번째로 google 소셜로그인 하여 police/insert 로 가셔서 admin으로 User생성 후 테스트 하시면 되겠습니다. 아직 수정할 부분이 더 있지만 member&security가 너무 지체되어 , 파트별로 테스트는 가능하게끔 좀 더 수정하였습니다. pull 하셔서 테스트 후 문제 생길 시 말씀주시면 되겠습니다.

Devinky commented 1 year ago

03/28 오후 4:15

사건 로직과 페이지 요소들 정상적으로 작동하는지 테스트 거친 후 merge 했고, 사건 로직이 추가됨에 따라 WebSecurity 클래스에 "/event/**" 경로를 추가했습니다

그리고 먼저 merge 해주신 김득주 이지창 팀원분들이 작업하신 부분에서 약간의 수정이 필요한 부분이 있었는데요

leftWrap(왼쪽 메뉴바)의 css가 따로 작성돼있지 않고 index.css로 적용돼있던 문제 -> 따로 leftWrap.css 작성해서 css/common 폴더에 넣어놓고 기존에 있던 police 관련 페이지들의 css경로도 수정해놨습니다

jichang-lee commented 1 year ago

0330 10:37 merge 브런치에 캘린더 api 추가하였습니다.(기존에 있던 캘린더 엔티티& service,controller,dto 및 필요한 html,css 등 추가) 헌데 ,캘린더 서비스상에 문제가 좀 있습니다.

캘린더 일정 추가 시 같은 일정이 2개가 추가되는 문제
캘린더 일정 추가 시 연관되어있는 컬럼 police_id가 같이save 안되는 문제 현재 확인되는 문제는 이 두개로 보이는데 , 서비스 수정하고 더블체크하여 더 살펴보겠습니다. 수정하는대로 다시 말씀드리겠습니다.

ckdtls1124 commented 1 year ago

0330 10:37 merge 브런치에 캘린더 api 추가하였습니다.(기존에 있던 캘린더 엔티티& service,controller,dto 및 필요한 html,css 등 추가) 헌데 ,캘린더 서비스상에 문제가 좀 있습니다.

캘린더 일정 추가 시 같은 일정이 2개가 추가되는 문제

캘린더 일정 추가 시 연관되어있는 컬럼 police_id가 같이save 안되는 문제 현재 확인되는 문제는 이 두개로 보이는데 , 서비스 수정하고 더블체크하여 더 살펴보겠습니다. 수정하는대로 다시 말씀드리겠습니다.

policeId를 넣는 문제는 제가 해결해놓았습니다.

jichang-lee commented 1 year ago

index.html에 main-grid>ul>li에 "오늘 일정" 이라는 List를 만들었습니다. 구조적으로나 여러방면에서 어느정도 모양이 같아야 할텐데, 제꺼 한번 확인해주시고 문제없다면 이대로 merge-test에 push 하려고 합니다

대충 화면에 보여지는 모양은 이렇습니다 ( 코드는 jccc23 branch에서 봐주세요 )

ckdtls1124 commented 1 year ago

index.html에 main-grid>ul>li에 "오늘 일정" 이라는 List를 만들었습니다.

구조적으로나 여러방면에서 어느정도 모양이 같아야 할텐데,

제꺼 한번 확인해주시고 문제없다면 이대로 merge-test에 push 하려고 합니다

대충 화면에 보여지는 모양은 이렇습니다 ( 코드는 jccc23 branch에서 봐주세요 )

확인했습니다. 저는 올려주신 디자인을 응용해서 각자 맡은 부분을 보여주는 것도 나쁘지 않다고 생각합니다.

ckdtls1124 commented 1 year ago

Refactoring 관련하여, 간단한 안내문을 master branch의 Read.me 파일에 올려놓았습니니다. 내용 확인해 주시고, 더 나은 방향으로 수정이 필요하다고 생각하시면 주저하지 마시고 말씀부탁드립니다. Refactoring 기한은 4/24까지 입니다.

ckdtls1124 commented 1 year ago

이지창 : 회원 CRUD, 허인경 : 사건사고 CRUD, 김득주 : 로그인 & 시큐리티, 장효선 : 게시판 CRUD, 강창신 : 근태 CR, 민원 RU, 결재CRUD, mainIndex

챗봇 - 강창신, 허인경, 장효선 캘린더 - 장효선, 이지창 네이버 API 사용 - 강창신, 김득주

역할 분담 업데이트입니다.

이지창 : CICD 허인경 : 경찰철 분실물 조회 API 사용, 날씨 API 구현 장효선 : 영화 API 구현 강창신 : 영화 API 챗봇에 구현 김득주 : 버스 API 구현

ckdtls1124 commented 1 year ago

master 브랜치에 최종 파일을 올리려고 합니다. READ.me 파일 내용 중, 각 이름을 각자의 브랜치로 연결하고자 합니다. 따라서 하기 내용 확인하시고 댓글로 써주시면 되겠습니다.

필수 사항 : 프로필 사진, github 링크 선택 사항 : groupware 프로젝트에서 구현한 기능으로 연결되는 github링크

예시 :

20230425_124916