buildyourteam / Backend

Eskimo RestfulApi repo
0 stars 0 forks source link

refactor: auth refactoring #134

Closed always0ne closed 4 years ago

always0ne commented 4 years ago

주요 변경점(클라이언트가 알아야 할 부분)

구현 변경점

기타

@KimJinYounga @dev-kimdoyoung @inho2736 @osydoo

Resolves: #130

KimJinYounga commented 4 years ago

Java 표준 주석 적용 매우 좋은 것 같아. 대신 우리 현재 있는 모든 컨트롤러에 하나하나 다 적어야해 다싴ㅋㅋㅋ 하게 되면 이슈로 빼자

always0ne commented 4 years ago

Java 표준 주석 적용 매우 좋은 것 같아. 대신 우리 현재 있는 모든 컨트롤러에 하나하나 다 적어야해 다싴ㅋㅋㅋ 하게 되면 이슈로 빼자

이게 좀 좋은게 IDE에서 커서만 올려도 주석 설명이 뜨고, JavaDoc도 자동생성 시킬 수 있더라고... 이거도 엄청 노가다여가지고 아주 조심스럽게 제안해본거였어 만약 한다면 본인이 짠 코드 직접 적는게 좋을거 같아

KimJinYounga commented 4 years ago

Java 표준 주석 적용 매우 좋은 것 같아. 대신 우리 현재 있는 모든 컨트롤러에 하나하나 다 적어야해 다싴ㅋㅋㅋ 하게 되면 이슈로 빼자

이게 좀 좋은게 IDE에서 커서만 올려도 주석 설명이 뜨고, JavaDoc도 자동생성 시킬 수 있더라고... 이거도 엄청 노가다여가지고 아주 조심스럽게 제안해본거였어 만약 한다면 본인이 짠 코드 직접 적는게 좋을거 같아

좋지! 우리근데 리팩토링 하면서 안짠 코드도 많이 여기저기 건드리긴했는데 기준은 처음 코드 직접 작성한 사람으로 해야하나??

always0ne commented 4 years ago

Java 표준 주석 적용 매우 좋은 것 같아. 대신 우리 현재 있는 모든 컨트롤러에 하나하나 다 적어야해 다싴ㅋㅋㅋ 하게 되면 이슈로 빼자

이게 좀 좋은게 IDE에서 커서만 올려도 주석 설명이 뜨고, JavaDoc도 자동생성 시킬 수 있더라고... 이거도 엄청 노가다여가지고 아주 조심스럽게 제안해본거였어 만약 한다면 본인이 짠 코드 직접 적는게 좋을거 같아

좋지! 우리근데 리팩토링 하면서 안짠 코드도 많이 여기저기 건드리긴했는데 기준은 처음 코드 직접 작성한 사람으로 해야하나??

리팩토링 됐어도 로직 자체를 건드리진 않고, 메소드 쪼개고 그런식으로 한거니깐 그렇게 해도 좋을거 같아

KimJinYounga commented 4 years ago

Java 표준 주석 적용 매우 좋은 것 같아. 대신 우리 현재 있는 모든 컨트롤러에 하나하나 다 적어야해 다싴ㅋㅋㅋ 하게 되면 이슈로 빼자

이게 좀 좋은게 IDE에서 커서만 올려도 주석 설명이 뜨고, JavaDoc도 자동생성 시킬 수 있더라고... 이거도 엄청 노가다여가지고 아주 조심스럽게 제안해본거였어 만약 한다면 본인이 짠 코드 직접 적는게 좋을거 같아

좋지! 우리근데 리팩토링 하면서 안짠 코드도 많이 여기저기 건드리긴했는데 기준은 처음 코드 직접 작성한 사람으로 해야하나??

리팩토링 됐어도 로직 자체를 건드리진 않고, 메소드 쪼개고 그런식으로 한거니깐 그렇게 해도 좋을거 같아

좋아 처음에 나눌때도 큼직큼직하게 나눠서 패키지 단위일듯!! 이번 회의 때 각자짠 코드 정리해보고 작업 들어가자!

dev-kimdoyoung commented 4 years ago

머지합시다~!!!