-
Notifications
You must be signed in to change notification settings - Fork 0
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
#140 : 인증된 사용자 테스트 #141
Open
hymi54
wants to merge
123
commits into
DSM-131
Choose a base branch
from
DSM-140
base: DSM-131
Could not load branches
Branch not found: {{ refName }}
Could not load tags
Nothing to show
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
#140 : 인증된 사용자 테스트 #141
Conversation
This file contains 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
테이블 행 삭제 시 다이얼로그에 있던 두 가지 옵션을 없애고 메타 데이터와 Dicom을 동시에 삭제하는 것만 가능하도록 변경합니다. 삭제 다이얼로그에서는 옵션 선택이 아닌 확인 기능만을 수행합니다. Task: #130
1. Collapsible Table인 Study UID Table을 삭제합니다. 2. Dicom Table Row에서 펼치기 버튼(화살표 아이콘)을 삭제합니다. Task: #130
프로젝트 드로어에서 Non-Reference Dicom 카테고리를 보이지 않게 합니다. 추후 코드 수정을 위해 지우지 않고 주석 처리만 합니다. Task: #130
기존에 테이블 행의 어느 곳이나 클릭해도 행 선택이 가능하던 것을 체크박스를 클릭했을 때에만 행 선택이 가능하도록 변경합니다. Task: #130
테이블 헤드 첫 번째 셀의 padding을 테이블 행 첫 번째 셀과 동일하게 설정합니다. Task: #130
기존에 Patient ID를 기준으로 Dicom 삭제 및 다운로드했던 것을 이제 Study UID를 기준으로 Dicom 삭제 및 다운로드하도록 새로운 api를 연결합니다. Task: #130
메인 페이지에 진입하기 전, 로그인 여부를 확인할 때 사용자에게 보여줄 로딩 페이지를 추가했습니다. Task : #136
RedirectionController에서 항상 imdc.hopto.org로 리다이렉션되었던 버그를 동적으로 적용가능하도록 수정함
이제 페이지가 시작할 때, 로그인을 검사 기능이 수행됩니다. Task : #136
로그인 기능이 페이지 전에 구현되어 있기 때문에, 기존에 사용하던 로그아웃 버튼으로 대체하였습니다. 또한, 로그아웃 이후 다시 메인페이지로 돌아오기 위한 코드를 벡엔드에서 수정했습니다. Task : #136
#136 - 로그인 확인 기능 구현
테이블 행 클릭 시 OHIF 뷰어로 리다이렉트합니다. Task: #130
LoadingPage의 Dialog 컴포넌트에서 open prop에 boolean type이 아닌 string type을 전달하여 에러가 발생하였으므로 이를 수정합니다. Task: #130
Fix : set server configure local to prod
프로젝트가 삭제되었을 때, getNonReferenced() 메서드가 호출되어 참조되지 않는 데이터가 보여지는 버그를 수정
기존에 사용하던 getNonReferenced 함수가 더이상 필요없어 주의 메시지기 출력되므로 임시로 주석처리
이제 프로젝트가 존재 하지 않는다면, 자동으로 프로젝트 생성 다이얼로그가 열리게 됩니다.
이제 프로젝트 정보를 가져올 때, 로딩페이지로 전환되며 프로젝트 정보를 가져온 이후에 메인 페이지에서 랜더링을 진행합니다.
이제 새로운 프로젝트를 생성하면 갱신된 정보가 반영된 프로젝트 리스트가 열립니다.
인증된 사용자를 사용한 테스트를 진행하는 도중 오류가 생겨 리뷰를 위해 커밋
#140 이슈에서 언급했든, WebMvcTest는 Controller 레이어에 있는 리소스만 가져오기 때문에, Spring Security관련 Configuration을 가져오진 않지만 빠르다. 그러나 현재 테스트에서는 Spring Security내용이 필요하기 때문에 속도를 희생하는 대신, @SpringBootTest로 전환하였다. Resolved : #140
이전 커밋에서 누락 정보 추가. 테스트 리소스에도 프로파일 정보가 추가되어야 한다. Resolved : #140
#180 프로젝트 초대, 취소 그리고 초대자들의 리스트를 반환하는 API 작성
1. 프로젝트에 초대된 유저 삭제 버튼을 툴바에 추가합니다. 2. Invite 폴더명을 InviteUser로 변경합니다. Task: #177
1. 프로젝트에 이메일로 유저를 초대하는 api를 뷰와 연결합니다. 2. 유저 삭제 다이얼로그 틀을 생성합니다. Task: #177
다른 사용자에게 초대 받은 프로젝트를 불러오는 api를 뷰와 연결합니다. Task: #175
내가 생성한 프로젝트와 다른 사용자에게 초대 받은 프로젝트를 구분하기 쉽도록 프로젝트 드로어에 카테고리 이름을 추가합니다. Task: #175
#175 - Drawer에서 본인 생성 프로젝트와 초대 받은 프로젝트 분리
프로젝트에 이미 초대한 사용자 이메일을 초대 다이얼로그에서 입력하는 것을 막습니다. Task: #177
issue : #180 기능 변경 및 추가 : 1. @PutMapping ("api/Project/{projectId}/oust/list") - 일반적으로 프로젝트 생성자가 방문자 삭제 시 사용되며, 생성자는 여러 사용자의 email을 담아 리스트로 보내면, 해당 사용자가 방문자 목록에서 삭제된다. 2. @PutMapping ("api/Project/{projectId}/oust") - 본인이 프로젝트 초대 목록에서 나가고 싶을 때 사용하는 api 이다, session을 통해 유저 이메일을 찾아 내 삭제 하는 방식으로 진행하였다.
Feat : 방문자 삭제 API 세분화
프로젝트에 초대된 유저를 내보내는 다이얼로그를 추가합니다. 삭제하고자 하는 유저를 목록에서 선택할 수 있습니다. Task: #177
1. 본인이 생성한 프로젝트에서 다른 사용자를 내보내는 api를 뷰와 연결합니다. 2. 사용자 초대 다이얼로그에서 추가한 이메일이 존재하지 않는 경우 초대 버튼 클릭을 막습니다. 3. 사용자 삭제 다이얼로그에서 초대된 이메일이 존재하지 않는 경우 삭제 버튼 클릭을 막습니다. Task: #177
1. 초대 받은 프로젝트에서는 툴바에서 사용자 삭제 버튼 대신 나가기 버튼이 보여집니다. 2. 나가기 버튼 클릭 시 나가기 다이얼로그를 띄웁니다. 3. 초대 버튼은 프로젝트 생성자에게만 보이도록 수정합니다. Task: #177
초대 받은 프로젝트에서 나가는 api를 뷰와 연결합니다. Task: #177
#177 - 프로젝트 초대 버튼 및 다이얼로그 생성
사용자 삭제 다이얼로그에서 아무 이메일도 선택하지 않았을 때 삭제 버튼 클릭을 막습니다. Task: #188
issue : #187 기능 추가 : 이슈 태그에 구성 된 것 과 같이 성공 시 Response 를 구성함
사용자 삭제 다이얼로그에서 취소 버튼 클릭 시 선택했던 이메일들을 체크 해제합니다. Task: #188
Fix : update submodule package version
issue : #187 기능 추가 : 프로젝트 방문자 초대 및 삭제 시 이메일 중 존재하지 않는 사용자에 대한 에러 응답으로 Error Code 400 : Bad Request 를 전달해주면서 message로 존재하지 않는 이메일 목록의 list를 준다.
issue : #187 기능 변경 : Project Invite,Oust API 호출 시 존재하지 않는 유저 목록을 failList로 보내주는 형식으로 변경함
#187 - 서버 Response Entity 구성
1. response 구조를 수정한 api를 뷰와 연결합니다. 2. 사용자 초대, 삭제 실패 시 서버에서 보내는 error message와 error failList를 프론트에서 alert로 띄워 사용자에게 요청 실패를 알립니다. Task: #188
#188 - 사용자 초대 관련 에러 핸들링
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.
인증된 사용자를 사용한 테스트를 진행하는 도중 오류가 생겨 리뷰를 위해 커밋