Skip to content

[BE] temp: 데모용 기능 브랜치를 확인한다.#473

Open
soominsohn wants to merge 27 commits intodemo-prfrom
demo
Open

[BE] temp: 데모용 기능 브랜치를 확인한다.#473
soominsohn wants to merge 27 commits intodemo-prfrom
demo

Conversation

@soominsohn
Copy link
Collaborator

@soominsohn soominsohn commented Oct 14, 2022 •
edited by dongho108
Loading

Issues

논의사항

  • 머지 안하고 데모 끝나면 PR Close 하겠습니다!
  • 브랜치 아무거나로 날렸습니다!

10/15 변경사항

demo 로그인, 데이터 방식 살짝 변경했습니다!

  1. 100명 코치, 크루 만들어놓고 랜덤으로 줍니다.
  2. 인터뷰 더미데이터도 만들어 놨습니다.

close #472

@soominsohn soominsohn changed the base branch from refactor-interview to demo-pr October 14, 2022 01:17
@dongho108 dongho108 changed the title [BE] feat: 데모용 기능 브랜치를 확인한다. [BE] test: 데모용 기능 브랜치를 확인한다. Oct 14, 2022
@dongho108 dongho108 changed the title [BE] test: 데모용 기능 브랜치를 확인한다. [BE] temp: 데모용 기능 브랜치를 확인한다. Oct 14, 2022
dongho108
dongho108 requested changes Oct 14, 2022
View reviewed changes
Copy link
Collaborator

@dongho108 dongho108 left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

앤지 고생하셨어요! demo용 코드 잘 봤습니다~

데모서버라 코드스타일같은 부분은 하나도 안보고 동시성 문제관련 코멘트 하나 남겼습니다~

논의 할 점은

  1. 저희가 데이터를 크루 100명, 코치100명 데이터를 만들어놓고 차례대로 주는데
    동시에 크루, 코치 100명이 넘게 접근하면 같은 계정을 쓸 수도 있을 것 같아요.
    -> 인원제한을 좀 많이 늘려놓을까요?

  2. 일주일동안 열고 데모기간에도 열면 혹시 코치의 면담가능시간이 부족할 수도 있을 것 같은데 어떻게 할까요?
    -> 최대한 많이 열어놓을까요?

  3. 코치 임시id로 로그인 하는 사람들도 면담스케줄을 볼 수 있게 하려면 어떻게 해야할까요?
    -> 더미데이터를 넣어놔야 하나..

backend/src/main/java/com/woowacourse/ternoko/auth/application/AuthService.java Outdated
Comment on lines 45 to 46
private static long COACH_ID = 0L;
private static long CREW_ID = 0L;
Copy link
Collaborator

@dongho108 dongho108 Oct 14, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r: 멀티쓰레드 환경에서는 AtomicLong 을 사용해야할 것 같아요

Copy link
Collaborator

@dongho108 dongho108 Oct 15, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

반영완료

@dongho108
Copy link
Collaborator

dongho108 commented Oct 14, 2022 •
edited by soominsohn
Loading

  1. 토큰 브라우저 캐싱문제 [데모]
    로그아웃, 홈으로 버튼 만들기

  2. 인터뷰 더미데이터 [데모]

  • availableDateTime 없이 생성됨
  1. 코멘트 문제 [운영]
  • 코멘트 없는 인터뷰여도 id가 있어서 코멘트 수정으로 가는 문제 - 운영에도 문제있음

@HyeonbinSa
Copy link
Collaborator

HyeonbinSa commented Oct 14, 2022 •
edited
Loading

  1. 코멘트 수정 시 스낵바 문구에서 오타 발견 [운영]
    • 정삭적으로 -> 정상적으로
  2. 회원 정보 수정 시 닉네임이 중복될 경우 알림 메세지로 보여줘야할듯 [운영]
    • 현재 500에러만 반환하고 버튼이 동작하지 않음.
    • 백엔드 머지하면 OK,
    • 프론트엔드에서 에러처리 추가 필요
  3. 10월 한눈에 보기 클릭 시 9월 달력 출력됨 [운영]

image

  1. 완료된 면담 삭제 불가
    • 완료된 면담이 삭제를 할 수 없다면 삭제 버튼을 지우는게 좋을 것 같습니다.
    • 완료된 면담을 삭제할 수 있다면 삭제할 수 있게 처리해야할 것 같습니다.

@her0807
Copy link
Collaborator

her0807 commented Oct 14, 2022 •
edited by soominsohn
Loading

Comment id cashing 문제 [운영]

완료된 면담으로 갔을 때, 코멘트 id 가 있으면 -> 캐싱 된 상태로 진행중 면담에 코멘트가 수정 버튼으로 보인다.

@dongho108
Copy link
Collaborator

dongho108 commented Oct 14, 2022 •
edited by soominsohn
Loading

N월 한눈에 보기 -> 9월로 고정되어있음 [운영]

@her0807
Copy link
Collaborator

her0807 commented Oct 14, 2022 •
edited by soominsohn
Loading

닉네임 수정 시

중복 아이디 일 경우 오류 안내를 해야하는데, 안내가 안되고 있다. [운영]

@dongho108
Copy link
Collaborator

dongho108 commented Oct 14, 2022 •
edited
Loading

완료된 인터뷰삭제 불가 [운영]

sanaandmomo and others added 20 commits October 14, 2022 18:33
@sanaandmomo
[FE] refactor: 코치 시간 생성 달력과 크루 시간 선택 달력을 분리한다. (#458)
7683d14 
* refactor: 코치 시간 생성 캘린더 분리

* refactor: 크루 인터뷰 신청 캘린더 분리

* refactor: 코치 시간 생성 캘린더와 크루 인터뷰 신청 캘린더를 children 패턴으로 리팩터링

* refactor: 스타일 로직 분리

* refactor: 필요 없어진 calendar 컴포넌트 삭제

* refactor: 코치 스케줄 캘린더 children 패턴으로 리팩터링

* refactor: 캘린더 관련 필요 없는 파일 제거

* refactor: 캘린더 month와 monthIndex 구분

* refactor: setDay -> removeDate, addOrSetDateBySelectMode 로 분리

* refactor: isSameDate 유틸 함수 활용으로 더 가독성 높이기

* fix: 날짜 범위 버그

* refactor: common 폴더 안에 있는 파일 common 접두사 제거

* refactor: CoachScheduleCalendar -> memo로 감쌈

* refactor: ModalPositionType -> BabyModalPositionType 네이밍 변경

* refactor: isDateNotInOfSelectedDatesAfterToday 함수 네이밍 변경
@soominsohn
[BE] refactor: 중복된 멤버 이름으로 닉네임 수정 시 검증 로직 추가 (#452)
862548d 
* refactor: 메서드 이름 변경

* refactor: partUpdateCoach 메서드 중복 닉네임 검증 로직 추가

* refactor: partUpdateCrew 메서드 중복 닉네임 검증 로직 추가

* refactor: 닉네임 중복 로직 JPQL로 변경

* refactor: partUpdateCoach 메서드 명명 updateCoach로 변경

* refactor: partUpdateCrew 메서드 명명 updateCrew로 변경
@her0807
[BE] Custom exception 을 리팩터링한다. (#463)
b6a0a33 
* refactor: coach custom exception 처리 정리

* refactor: crew custom exception 처리 정리

* refactor: interview custom exception 처리 정리

* refactor: comment custom exception 처리 정리

* refactor: AvailableDateTime custom exception 처리 정리

* refactor: Member 관련 custom exception 처리 정리

* refactor: Token 관련 custom exception 처리 정리

* refactor: exception package 변경

* refactor: 불필요한 exception 제거
@HyeonbinSa @dongho108
[BE] refactor: @builder 어노테이션 buildMethodName 속성 제거 (#467)
bcbb841 
* refactor: 인터뷰 픽스처 추가 및 테스트코드 리팩터링

* refactor: 깨지는 테스트 해결 및 메서드 리팩터링

* refactor: Builder 어노테이션 불필요 buildMethodName 속성 제거

Co-authored-by: dongho <dongho108@naver.com>
@Juhyung990122 @sanaandmomo @soominsohn
[BE] docs: 데모용 리드미 작성 (#482)
170ad0e 
* [FE] feat: 코치 되는 시간 생성시, 주간 선택을 지원한다.  (#460)

* feat: 코치 되는 시간 생성시, 주간 선택을 지원한다

(cherry picked from commit fd8f0977f34d5c7e327a0a7a9cf26029095add40)

* chore: conflict 해결

* refactor: onChangeSelectedDatesByClickDayOfWeek -> onChangeDateLine 으로 함수 네이밍 변경

(cherry picked from commit df60d19c83f7753891a4c219a20b373fec93402a)

* refactor: 중복되는 로직을 고차함수를 통해 제거

(cherry picked from commit 44099964ed480c44705cd39696bf4c8cb67f56f0)

* refactor: 라인이 다 selected 여부인지 확인하는 함수 재활용

(cherry picked from commit 0bfac646ae6e263563d02fff22d4718da421d721)

* fix: row cell을 forEach 하는 범위 버그

(cherry picked from commit cfca3b45f3a05e4c18f81748c193f975d2de5eba)

* refactor: of 전치사 제거

* docs: readme 초안 작성

* docs: readme 이미지 추가

* docs: gif추가

* docs: update 수정사항 반영

* docs: readme gif 재생 조절

* docs: 기술스택, 아키텍처 업로드

* docs: 이미지 크기 조정

* docs: 리드미 사진중복 수정

* docs: 리드미 사진 포멧 수정

Co-authored-by: soobin <38878617+sanaandmomo@users.noreply.github.com>
Co-authored-by: Soomin Sohn <soomin.sohn.dev@gmail.com>
@her0807
chore: 모니터링 설정 추가 (#485)
29fcfcb
@lokba
[FE] QA를 진행하다 나온 버그를 수정한다. (#476)
ac2c360 
* chore: 코멘트 문구 수정

(cherry picked from commit 8cb6d4ffbe52a83bc50b21e89a9c8142567b382e)

* feat: 완료한 면담의 경우 삭제 불가능

(cherry picked from commit b801c8d56288ad9e48b35465813d99f3d0ddfd75)

* refactor: 불필요한 import 구문 삭제

(cherry picked from commit 83606e8d35ba3a1d49a0744c238ffa20e1a03e6b)

* refactor: 닉네임 중복시, 에러 메세지 띄우기

(cherry picked from commit 400690f9bb41544596cddbc0a9ee821a4a9bd160)

* fix: 코멘트id 초기화 로직 추가

(cherry picked from commit d3333970e03c1f49bd06793b56fe553d7053e8d2)

* refactor: convert isAbleDelete to canDelete
@lokba
[FE] refactor: 디렉토리 구조를 개편한다. (#489)
155e90e 
* refactor: arrange directory structure based on domain

* refactor: rearrange directory structure based on domain
@dongho108
Release v1.2.2
28215b2 
Release v1.2.2
@soominsohn @dongho108
feat: 메일기능 주석처리, 슬랙알람기능 주석처리, 크루로그인 코치로그인 생성
a66a370
@soominsohn @dongho108
refactor: DatabaseInitializer profile deomo 추가
a71b9d2
@soominsohn @dongho108
refactor: DatabaseInitializer profile deomo 추가
799cc6e
@soominsohn @dongho108
refactor: DemoDatabaseInitializer profile mockData 추가
d5d4f75
@soominsohn @dongho108
refactor: available_dat_time mock data 추가
703b005
@soominsohn @dongho108
refactor: crewId 증가 로직 변경
4093dcd
@dongho108
refactor: Atomic Long 으로 변경
04f7ad0
@dongho108
fix: crewId 제대로 시작하도록 설정
2f10563
@soominsohn @dongho108
refactor: COMMENT상태 인터뷰 등록
210c4b3
@soominsohn @dongho108
refactor: 인터뷰 상태 변경
911f818
@soominsohn @dongho108
refactor: 인터뷰 mock data 추가
6d4beed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dongho108 dongho108 dongho108 requested changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@soominsohn @dongho108 @HyeonbinSa @her0807 @sanaandmomo @Juhyung990122 @lokba