forked from woowacourse/miniprojects-2019
-
Notifications
You must be signed in to change notification settings - Fork 5
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
8/15 게시글 코드리뷰 반영 #55
Labels
Comments
dasistHYOJIN
added a commit
that referenced
this issue
Aug 17, 2019
JunHoPark93
pushed a commit
that referenced
this issue
Aug 20, 2019
* init: project 생성 * init: 프론트 뷰 파일 초기 세팅 * [#18] feat: 공통 패키지 및 엔티티 정의 * [#15] feat: 게시글 모델 생성 1. Article 엔티티 클래스를 추가한다 2. Article 엔티티의 하위 VO 클래스를 추가한다 3. 그래들에 의존성을 추가한다 * [#7] feat: Member 엔티티 정의 * [#20] refactor: index.html의 눈 모양 아이콘을 게시글 작성 아이콘으로 변경 * [#20] feat: article을 작성하는 기본적인 페이지 작성 * [#20] feat: article-edit.html의 자바스크립트 틀 작성 * [#8] feat: 회원가입 기능 Closes #8 * [#15] feat: 게시글 작성 기능 구현 1. 게시글을 작성하는 기능을 추가한다 1-1. ArticleController 클래스를 추가한다 1-2. POST 요청 시, 작성된 게시글을 저장한다 1-3. ArticleRequest DTO 클래스를 추가한다 1-4. ArticleRepository 클래스를 추가한다 2. 게시글 작성 기능에 대한 컨트롤러의 테스트 메서드를 추가한다 * [#22] refactor: 사용하지 않는 js console.log 삭제 * [#26] refactor: 8월 13일 리뷰에 따른 리팩토링 적용 * [#22] feat: zzazanstagram logo navBar에 추가 및 다른 페이지에도 적용 * [#29] refactor: 8월 13일 게시글 관련 리뷰에 따른 리팩토링 적용 * [#30] feat: 게시글 작성 페이지와 서버 시스템 연결 * [#10] feat: 로그인 기능 Closes #10 * [#15] feat: 게시글 조회 기능 구현 1. ArticleAssembler 클래스에 DTO로 변환하는 메서드를 추가한다 2. ArticleController에서 서비스 로직을 분리한다 3. 불필요한 어노테이션을 삭제한다 * [#31] feat: 게시글 조회 페이지와 서버 시스템 연결 * [#10] refactor: 테스트 중복 제거 * [#22] feat: signup 페이지 뷰 작성 * [#39] refactor: 8/14 리팩토링 * [#39] refactor: 8/14 리팩토링 * [#41] 회원가입 form의 placeholder 한글로 변경 및 정규식 pattern 추가 * [#41] 회원가입 페이지의 배치 정밀 수정 * [#41] feat: validate error 메시지 tooltip으로 추가 * [#24] feat: login.html 페이지 뷰 작성 * [#44] refactor: domain entity에 column 어노테이션 추가 * [#44] refactor: 예외 관련 테스트 케이스 추가 및 관련 프로덕션 로직 추가 * [#44] refactor: LoginController, LoginService를 Member관련 클래스에서 분리 * [#44] refactor: Article entity에 Member entity 연관관계 매핑 추가 * [#44] refactor: 프론트엔드 로직과 백엔드 로직 연동 * [#44] refactor: Member package 구조 변경 * [#44] refactor: MemberTest에 assertThat import * [#50] fix: 사용하지 않는 코드 제거 및 의도하지 않은 오타 등의 코드로직 수정 * [#50] refactor: html 태그에 각각 적용된 style 속성을 별도의 css 파일로 분리:zzazan.css * [#52] refactor: 8/15 피드백 반영 * [#52] refactor: 8/15 피드백 반영 dto에서 검사 * [#55] refactor: 8/15 게시글 피드백 반영 * [#57] refactor: 유틸 클래스(SessionKeys) 생성자 private 으로 선언 * [#57] feat: test 용 의존성 추가 기존에 gradlew 로 build 실행시에는 테스트를 skip하고 있어서 https://www.baeldung.com/junit-5-gradle 를 참고하여 의존성 추가 * [#57] refactor: Image 클래스에서 Http요청을 통해서 이미지 URL확인로직 제거 * [#57] refactor: RequestMapping을 활용하여 중복 Url제거 * [#57] refactor: validate 메서드 명 수정 * [#57] refactor: Email, NickName에 대해서 각각 요청하던 쿼리를 합침 기존의 existsByEmail, existsByNickName메서드를 통해서 각각 쿼리를 날리던 상황에서 existsByNickNameOrEmail를 사용하여 한 번만 쿼리를 날리도록 수정 * [#57] refactor: 메서드명 동사로 수정 Article, Member 클래스에서 Vo의 String 값을 요청하는 메서드명을 동사로 수정 * [#57] refactor: SessionKeys를 web package로 이동 * [#60] refactor: gradle test 추가
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
No description provided.
The text was updated successfully, but these errors were encountered: