[team-31][BE][Yan-K] To-do list 그룹 프로젝트 1주차(수) PR #23

KTH-96 · 2022-04-06T07:55:36Z

안녕하세요 리뷰어님^^ 팀 31 BE Yan, K 입니다.
오늘 까지 작업한 내용 PR드립니다. :)

구현사항

api설계 간단하게 작성했습니다 mapping방식은 다음번에 작성해서 올릴 예정입니다.
카드 저장, 삭제 로직 작성

어려웠던 부분

git으로 협업을 해보는 부분이 미숙하여 처음에 시간을 많이 잡아먹었습니다. :)

Update issue templates

closes #5

- domain, service, web 구조로 작성 close #5

closes #7

close #6

Min-92

@KTH-96 @leekm0310
안녕하세요. 얀, 케이 (제가 맞게 태그했나요?) 만나서 반갑습니다.

todo list 리뷰어를 맡은 왕민입니다.

이번 PR은 코멘트 남기고 머지하도록 하겠습니다.
남겨드린 코멘트 고민해보시고 다음 PR에 반영 또는 의견 부탁드려요 :)

git을 통한 협업이 어려우셨다면 어떻게 하면 좀 더 효율적일지, conflict 발생을 예방할 방법은 무엇일지 고민해보는것도 도움될듯합니다

Min-92 · 2022-04-06T15:02:07Z

todo-list/src/main/java/com/todolist/project/domain/card/Card.java

+
+import java.time.LocalDateTime;
+
+public class Card {


도메인 클래스 잘 작성해 주셨네요 👍

Min-92 · 2022-04-06T15:02:53Z

todo-list/src/main/java/com/todolist/project/domain/card/Card.java

+import java.time.LocalDateTime;
+
+public class Card {
+    private int id;


생성자에 id는 없어보이는데 id의 초기값은 어떻게 될까요?

id 값은 데이터 베이스에서 AUTO_INCREMENT를 사용할 생각입니다. :)

Min-92 · 2022-04-06T15:03:30Z

todo-list/src/main/java/com/todolist/project/domain/card/Card.java

+    private String title;
+    private String contents;
+    private String writer;
+    private LocalDateTime createdTime;


👍
dateTime 대신 localDateTime 사용해주신 부분 좋네요.
이미 알고계시겠지만, 왜 dateTime을 사용 안하는지 이유도 알고있으면 좋을듯 합니다.

일단 dateTime이 immutable객체가 아니라는점과 월을 지정할때 상수에서 -1씩 해줘야하는 불편함 때문인거 같습니다. :)

Min-92 · 2022-04-06T15:07:42Z

todo-list/src/main/java/com/todolist/project/domain/card/CardRepository.java

+    private final JdbcTemplate jdbcTemplate;
+    public CardRepository(DataSource dataSource) {
+        this.jdbcTemplate = new JdbcTemplate(dataSource);
+    }


생성자 주입을 통해서 JdbcTemplate을 주입해주셨네요 👍
다른 DI(Dependency Injection)방식에는 무엇이 있는지, 각 장단점은 어떤것인지 알고가면 좋을것 같습니다.

https://baek.dev/post/21/ 보면서 공부 해봤습니다. 감사합니다 :)

Min-92 · 2022-04-06T15:12:22Z

todo-list/src/main/java/com/todolist/project/domain/card/CardRepository.java

+    private final static String DELETE_CARD_SQL = "delete from card where id = ?";
+    private final static String INSERT_CARD_SQL = "insert into card(title, contents, writer, createTime, status) value (?,?,?,?,?)";


SQL은 대소문자를 구분하진 않지만, SQL 키워드는 대문자로 작성하는게 가독성이 좋습니다.
예약어와 테이블,컬럼명이 구분되면 어떤 장점이 있을까요?

다른 개발자가 보기에 짧은 시간에 구조를 파악 할 수 있는 장점이 있다고 생각합니다. :)

Min-92 · 2022-04-06T15:25:07Z

todo-list/src/main/java/com/todolist/project/web/CardController.java

+        return "redirect:/";
+    }
+
+    @DeleteMapping("/remove/{id}")


설계하신 uri가 'DELETE /card/remove/{id}' 인데요 뭔가 의미가 중복된 느낌이죠?
REST API URI규칙, 시맨틱 uri에 대해서 알아보세요 :)

넵 확인해서 수정 하도록 하겠습니다!!

Min-92 · 2022-04-06T15:25:20Z

todo-list/src/main/java/com/todolist/project/web/CardController.java

+        this.cardService = cardService;
+    }
+
+    @PostMapping("/add")


아래 DELETE와 동일한 내용

Min-92 · 2022-04-06T15:29:58Z

todo-list/src/main/java/com/todolist/project/web/dto/addCardDto.java

+    public addCardDto(String title, String contents, String writer) {
+        this.title = title;
+        this.contents = contents;
+        this.writer = writer;
+    }


클래스명이 동사로 시작하다보니 생성자가 마치 일반 메소드인듯한 착각이 드네요!

Min-92 · 2022-04-06T15:32:39Z

todo-list/src/main/java/com/todolist/project/domain/Status.java

@@ -0,0 +1,11 @@
+package com.todolist.project.domain;
+
+public enum Status {


Status는 방대한 의미를 가진듯 해요.
이 enum 클래스의 의미에 맞는 적절한 이름을 찾아보세요

수정 하도록하겠습니다!!

Min-92 · 2022-04-06T15:33:57Z

todo-list/build.gradle

+	compileOnly 'org.projectlombok:lombok'
+	annotationProcessor 'org.projectlombok:lombok'


롬복을 사용하지 않으면 불필요한 의존성으로 보이네요

제가 프로젝트 만들때 집어넣었나보네요 ㅠㅠ 의존성 삭제 하도록 하겠습니다 !!

- application.yml에 h2 관련 설정 - TODO 테이블과 더미 데이터 생성 (schema.sql, data.sql) closes #21

[후] 카드(할일) 도메인 개발 리팩토링

* Update issue templates * docs: Update Readme Readme 1차 업데이트 * docs: Update Readme 기술 스택 추가 * Design: reset scss 적용 * HTML, CSS 작업 - Header, Sidebar (#11) * design: header HTML markup Related to: #8 * design: header css style Related to: #8 * design: sidebar HTML markup Related to: #8 * design: sidebar css style Related to: #8 * 메뉴 - 우측 히든 레이어 생성, 애니메이션 구현 (#12) * feat: sidebar 메뉴 버튼 사이드바 메뉴 버튼 클릭시 사이드바 숨김,보임 기능 구현 Related to: #10 * design: sidebar 애니메이션 왼쪽방향으로 나타나고 오른쪽 방향으로 숨겨지는 애니메이션 구현 Related to: #10 * 메뉴 - 우측 히든 레이어 생성, 애니메이션 구현 (#13) * feat: sidebar 메뉴 버튼 사이드바 메뉴 버튼 클릭시 사이드바 숨김,보임 기능 구현 Related to: #10 * design: sidebar 애니메이션 왼쪽방향으로 나타나고 오른쪽 방향으로 숨겨지는 애니메이션 구현 Related to: #10 * feat: action분리 action에 따라 sidebar 리스트의 comment를 다르게 출력함 Related to: #10 * feat: timestamp 기능 구현 현재 시간을 기준으로 투두리스트 작성 및 변경 시간을 보여주는 timestamp 구현 Related to: #10 * feat: 을/를, 으로/로 구분 단어의 마지막 글자 종성에 따라 모음을 다르게 렌더하는 기능 구현 Related to: #10 * HTML, CSS 작업 - Card List (#14) * design: main column list html markup Related to : #9 * move: svg file move to public/svg * 기존 public 에 있던 svg 파일을 svg 폴더 내로 이동 * icon-add.svg 파일 추가 Related to : #9 * Design: font mixin 함수 추가 Related to : #9 * design: column list scss style 작성 Related to: #9 * design: column task list scss style 작성 Related to: #9 * design: task commnet padding 을 margin 으로 변경 Related to: #9 * design: sccs convert to css Related to: #9 * fix: icon svg 파일 경로 수정 Related to: #9 * chore: 빌드환경구성 (#16) babel, webpack 적용 Related to: #15 * 메뉴 - 사용자 액션 리스트 구현 (#19) * feat: sidebar 액션 리스트 히든레이어 영역을 애니메이션 효과와 함께 숨김,보임 기능 구현 Related to: #17 * design: sidebar 액션리스트 애니메이션 Related to: #17 * move: package.json, gitignore * feat: sidebar 스크롤 기능 구현 액션리스트 기록이 많아지면 스크롤을 이용하여 다음 기록을 확인할 수 있음 Related to: #17 * design: 액션리스트 스크롤 구현 Related to: #17 * feature: json server 생성 및 db.json 추가 (#21) * json server 설치하고 db.json 에 필요한 데이터들을 추가함 Related to: #18 * HTML, CSS 작업 - Card (#22) * feat: todo-list 입력창 높이 자동 조절 todo-list를 입력하거나 지울 때 입력창의 높이를 자동으로 조절하는 기능 구현 Related to: #20 * design: column button hover color 칼럼 타이틀의 +,x 버튼을 hover할 때 버튼 색상 변경 Related to: #20 * design: 수정카드 디자인 구현 Related to: #20 * 리팩토링 - 1주차 리뷰 반영 (#24) * refactor: getFinalConsonant 매직넘버 제거 Related to: #23 * refactor: calcTimeForToday 함수 리팩토링 * 매직 넘버 제거 * DayDifference 식 수정 * 몇 년전 조건 추가 Related to: #23 * refactor: writeTime -> timeStamp Related to: #23 * refactor: identifyCategory 함수 리팩토링 * 구조 분해 할당 적용 Related to: #23 * refactor: svg 파일 경로 변수로 분리 * svg 파일 경로를 constants의 imagePath에 변수로 생성해서 관리하도록 변경 Related to: #23 * feat: fetchData 함수 구현 Related to: #23 * chore: bundle.js Related to: #23 * refactor: sidebar model 리팩토링 Related to: #23 * refactor: activationStore import 방식 변경 Related to: #23 Co-authored-by: Hemdi <[email protected]> Co-authored-by: Hemudi <[email protected]>

* Update issue templates * Docs: 이슈템플릿 수정 * Docs: pull request template 생성 * [SCSS 및 폴더구조 구성]13 htmlcss 정적 페이지 (#14) * Design: SCSS 초기 구조 세팅 * Refactor: views directory 구조 및 이름 변경 * Fix: dist 삭제 * Chore: .gitignore 수정 * 16 정적페이지 기초구성 (#18) * Docs: prettier 설정 * Design: header, card icon image 추가 * Design: header & column CSS 추가 * Design: index.html 추가 * Feat: 필요없는 파일 제거 * Refactor: 데이터 구조 변경 - 코드 포맷: 카멜로 변경 - 프로퍼티명의 중복 제거 : item.item_title -> item.title * 정적페이지 card,alert,layer (#19) * Design: icon svg 추가 * Design: 활동기록 html 추가 * Design: column, card, history css * 아직 못한 html&css - 카드생성박스 - 취소 Alert창 - 추가기능 html * Feat: 스토어 메소드 추가, 프로퍼티 네임 변경 - 스토어 프로퍼티 네임을 가져오는 데이터의 프로퍼티와 일치시킴 - 메소드 추가(메소드명만) * Refactor: 불러오는 유저 데이터 변수명 변경 storage -> userData * Refactor: 안쓰는 라인 삭제 * Refactor: 데이터구조변경으로 인한 유저찾는 로직 수정 * Refactor: 엔트리 함수 이름을 main -> app 변경 - 파일 이름과 맞춤 * Feat: 엘리먼트를 특정 위치에 추가하는 헬퍼 함수 작성 * Feat: column, item 엘리먼트를 생성하는 템플릿함수 작성 * Feat: 화면에 엘리먼트를 추가하는 렌더링 함수모음 객체 renderer 작성 * Feat: 페이지 로드 시, 컬럼과 아이템을 모두 렌더링하는 로직 추가 * Fix: 파일이름 오류 수정 * Feat: asideView 추가 (show,hide 동작) * Fix: 카드UI 위아래 붙는 오류 해결 * Feat: history 생성 함수 추가 * Feat: 현재 시각 대비 특정 시각 얻는 헬퍼함수 추가 * Feat: 히스토리 내역 데이터 추가 * Feat: 페이지 로드 시 히스토리 내역 렌더하기 * Refactor: 컨트롤러 구조 변경 기존의 클래스 -> 함수 * Refactor: 메소드 이름 변경 set -> update * Refactor: controller -> handler 이름 변경, app에 핸들러 엔트리 함수 추가 * Design: 남은 정적페이지 분량 (#23) * Design: 카드등록창 이미지 추가 * Design: Card 등록박스 임시로 추가 * Refactor: textarea rows 속성 추가 Co-authored-by: Sangbeom Heo <[email protected]>

[iOS] CardTableView와 CardCell 구현

[BE] 도메인 객체 추가 및 레이어 클래스 추가

MainVC와 InspectorVC를 관리하는 ContainerVC 생성

* 카드 등록 - 새로운 카드 등록 기능 구현 (#29) * feat: todolist column과 task를 스크립트로 렌더링 list view와 task view를 클래스로 생성하고 db.json의 todolist를 하나의 배열로 묶음 * feat: column add button 칼럼의 추가버튼으로 등록카드 생성,삭제 기능 구현 * feat: task cancle button 등록카드의 취소버튼을 누르면 등록카드를 삭제하는 기능 구현 * refactor: 메인 렌더링 영역을 list에서 main으로 수정 entry point에서 mainInit의 parent를 list에서 main으로 리팩토링 * refactor: input event를 메인에서 task 모듈로 분리 입력창을 autoResize하는 이벤트를 task 모듈로 분리함 * rename: index.js * 카드 등록 - 새로운 카드 입력창 생성 기능 구현(1) (#32) * feat: 등록버튼 활성화 기능 구현 등록카드에 내용을 입력하면 등록버튼을 활성화, 입력할 수 있는 최대 글자수는 500자 * refactor: registration activation을 todoListStore에서 관리 activiation관리를 task에서 todoListStore로 변경 * 카드 이동 - 드래그앤드랍으로 카드 이동 구현(1) (#33) * design: task cursor 변경 * task -> grab * delete button -> pointer * feat: mousedown 시 copyTask 생성 기능 구현 * Task 에 mousedown 이벤트 발생 시 드래그 할 복사본 Task 요소를 생성하는 기능 구현 * feat: index에서 todoList render 후에 dragEvent 등록 기능 구현 * 카드 등록 기능 구현 된 로직에 dragEvent 설정 로직 병합 * 그 외 자잘한 수정 포함 * feat: task 요소 좌표 설정 기능 구현 * 클릭 이벤트가 발생한 좌표로 task 요소의 좌표를 설정하는 로직 구현 * feat: drag 기능 구현 * body에 mousemove 이벤트 등록해서 이벤트가 발생할때마다 해당 이벤트의 좌표로 task 요소를 이동되도록 구현 * feat: mouseup 이벤트 발생 시 드랍 & 복사 task 요소 삭제 기능 구현 * 복사한 task 요소에 mouseup 이벤트 설정 * Design: column task list height, hidden 클래스 추가 * Design: 잔상 처리용 클래스 blur 추가 * feat: drag and drop 기능 구현 * 선택 된 origin task item , 드래그 되고 있는 task item, 이동 된 task item 3개의 요소를 활용 * remove: todolistStore.js * rename: todolistStore -> todoListStore * 카드 이동 - 드래그앤드랍으로 카드 이동 구현(2) (#34) * design: task cursor 변경 * task -> grab * delete button -> pointer * feat: mousedown 시 copyTask 생성 기능 구현 * Task 에 mousedown 이벤트 발생 시 드래그 할 복사본 Task 요소를 생성하는 기능 구현 * feat: index에서 todoList render 후에 dragEvent 등록 기능 구현 * 카드 등록 기능 구현 된 로직에 dragEvent 설정 로직 병합 * 그 외 자잘한 수정 포함 * feat: task 요소 좌표 설정 기능 구현 * 클릭 이벤트가 발생한 좌표로 task 요소의 좌표를 설정하는 로직 구현 * feat: drag 기능 구현 * body에 mousemove 이벤트 등록해서 이벤트가 발생할때마다 해당 이벤트의 좌표로 task 요소를 이동되도록 구현 * feat: mouseup 이벤트 발생 시 드랍 & 복사 task 요소 삭제 기능 구현 * 복사한 task 요소에 mouseup 이벤트 설정 * Design: column task list height, hidden 클래스 추가 * Design: 잔상 처리용 클래스 blur 추가 * feat: drag and drop 기능 구현 * 선택 된 origin task item , 드래그 되고 있는 task item, 이동 된 task item 3개의 요소를 활용 * remove: todolistStore.js * rename: todolistStore -> todoListStore * feat: set drag and drop event * 카드 등록 - 새로운 카드 입력창 생성 기능 구현(2) (#35) * feat: 등록버튼 활성화 기능 구현 등록카드에 내용을 입력하면 등록버튼을 활성화, 입력할 수 있는 최대 글자수는 500자 * refactor: registration activation을 todoListStore에서 관리 activiation관리를 task에서 todoListStore로 변경 * 카드 등록 - 새로운 카드 입력창 생성 기능 구현(1) (#30) * [team-19] Todo-list 1주차 #1 코드리뷰 요청 (#32) * Update issue templates * docs: Update Readme Readme 1차 업데이트 * docs: Update Readme 기술 스택 추가 * Design: reset scss 적용 * HTML, CSS 작업 - Header, Sidebar (#11) * design: header HTML markup Related to: #8 * design: header css style Related to: #8 * design: sidebar HTML markup Related to: #8 * design: sidebar css style Related to: #8 * 메뉴 - 우측 히든 레이어 생성, 애니메이션 구현 (#12) * feat: sidebar 메뉴 버튼 사이드바 메뉴 버튼 클릭시 사이드바 숨김,보임 기능 구현 Related to: #10 * design: sidebar 애니메이션 왼쪽방향으로 나타나고 오른쪽 방향으로 숨겨지는 애니메이션 구현 Related to: #10 * 메뉴 - 우측 히든 레이어 생성, 애니메이션 구현 (#13) * feat: sidebar 메뉴 버튼 사이드바 메뉴 버튼 클릭시 사이드바 숨김,보임 기능 구현 Related to: #10 * design: sidebar 애니메이션 왼쪽방향으로 나타나고 오른쪽 방향으로 숨겨지는 애니메이션 구현 Related to: #10 * feat: action분리 action에 따라 sidebar 리스트의 comment를 다르게 출력함 Related to: #10 * feat: timestamp 기능 구현 현재 시간을 기준으로 투두리스트 작성 및 변경 시간을 보여주는 timestamp 구현 Related to: #10 * feat: 을/를, 으로/로 구분 단어의 마지막 글자 종성에 따라 모음을 다르게 렌더하는 기능 구현 Related to: #10 * HTML, CSS 작업 - Card List (#14) * design: main column list html markup Related to : #9 * move: svg file move to public/svg * 기존 public 에 있던 svg 파일을 svg 폴더 내로 이동 * icon-add.svg 파일 추가 Related to : #9 * Design: font mixin 함수 추가 Related to : #9 * design: column list scss style 작성 Related to: #9 * design: column task list scss style 작성 Related to: #9 * design: task commnet padding 을 margin 으로 변경 Related to: #9 * design: sccs convert to css Related to: #9 * fix: icon svg 파일 경로 수정 Related to: #9 * chore: 빌드환경구성 (#16) babel, webpack 적용 Related to: #15 * 메뉴 - 사용자 액션 리스트 구현 (#19) * feat: sidebar 액션 리스트 히든레이어 영역을 애니메이션 효과와 함께 숨김,보임 기능 구현 Related to: #17 * design: sidebar 액션리스트 애니메이션 Related to: #17 * move: package.json, gitignore * feat: sidebar 스크롤 기능 구현 액션리스트 기록이 많아지면 스크롤을 이용하여 다음 기록을 확인할 수 있음 Related to: #17 * design: 액션리스트 스크롤 구현 Related to: #17 * feature: json server 생성 및 db.json 추가 (#21) * json server 설치하고 db.json 에 필요한 데이터들을 추가함 Related to: #18 * HTML, CSS 작업 - Card (#22) * feat: todo-list 입력창 높이 자동 조절 todo-list를 입력하거나 지울 때 입력창의 높이를 자동으로 조절하는 기능 구현 Related to: #20 * design: column button hover color 칼럼 타이틀의 +,x 버튼을 hover할 때 버튼 색상 변경 Related to: #20 * design: 수정카드 디자인 구현 Related to: #20 * 리팩토링 - 1주차 리뷰 반영 (#24) * refactor: getFinalConsonant 매직넘버 제거 Related to: #23 * refactor: calcTimeForToday 함수 리팩토링 * 매직 넘버 제거 * DayDifference 식 수정 * 몇 년전 조건 추가 Related to: #23 * refactor: writeTime -> timeStamp Related to: #23 * refactor: identifyCategory 함수 리팩토링 * 구조 분해 할당 적용 Related to: #23 * refactor: svg 파일 경로 변수로 분리 * svg 파일 경로를 constants의 imagePath에 변수로 생성해서 관리하도록 변경 Related to: #23 * feat: fetchData 함수 구현 Related to: #23 * chore: bundle.js Related to: #23 * refactor: sidebar model 리팩토링 Related to: #23 * refactor: activationStore import 방식 변경 Related to: #23 Co-authored-by: Hemdi <[email protected]> Co-authored-by: Hemudi <[email protected]> * 카드 등록 - 새로운 카드 등록 기능 구현 (#29) * feat: todolist column과 task를 스크립트로 렌더링 list view와 task view를 클래스로 생성하고 db.json의 todolist를 하나의 배열로 묶음 * feat: column add button 칼럼의 추가버튼으로 등록카드 생성,삭제 기능 구현 * feat: task cancle button 등록카드의 취소버튼을 누르면 등록카드를 삭제하는 기능 구현 * refactor: 메인 렌더링 영역을 list에서 main으로 수정 entry point에서 mainInit의 parent를 list에서 main으로 리팩토링 * refactor: input event를 메인에서 task 모듈로 분리 입력창을 autoResize하는 이벤트를 task 모듈로 분리함 * rename: index.js * feat: 등록버튼 활성화 기능 구현 등록카드에 내용을 입력하면 등록버튼을 활성화, 입력할 수 있는 최대 글자수는 500자 * refactor: registration activation을 todoListStore에서 관리 activiation관리를 task에서 todoListStore로 변경 Co-authored-by: Hemdi <[email protected]> Co-authored-by: Hemudi <[email protected]> * Revert "카드 등록 - 새로운 카드 입력창 생성 기능 구현(1) (#30)" (#31) This reverts commit 7155f85. * merge develop2 * feature: +버튼으로 등록 카드 생성 등록 카드가 이미 생성된 상황에서 다른 탭을 누르면, 기존의 카드가 사라지고 해당 탭에 새로운 카드를 생성 * feature: task에 이벤트 등록 task 인스턴스를 생성할때 이벤트 등록 Co-authored-by: Hemdi <[email protected]> Co-authored-by: Hemudi <[email protected]> Co-authored-by: mogooee <[email protected]>

Content - MainVc 의safe.trailingAnchor 에 활동기록뷰 의 leading 을 마추고 width 크기 지정 Todo - 뷰 애니메이션 기능 구현 - 테이블 뷰 구현

Content - 뷰 애니메이션 기능 구현 Todo - 활동기록 테이블뷰 구현

* Design: 카드등록창 이미지 추가 * Design: Card 등록박스 임시로 추가 * Refactor: textarea rows 속성 추가

Content - 기존 레이아웃 수정

Content - 어색한 애니메이션 수정 Todo - 활동기록 테이블뷰,셀 만들기

Content - 활동 기록 뷰 레이아웃 수정

* Chore: utils.js 추가 (#20) on, delegate, emit추가 * Feat: 컬럼에 카드 등록 진행 중 #21 (#22) * Fix: colors 오타 수정 gray4 두개 선언 -> gray5로 수정 * Refactor: utils.js 수정 on 함수 삭제 delegate -> eventDelegate로 이름 변경 eventDelegate 로직 변경: potentialElement가 event.target 대신, event.target.closest(selector)와 같은지 비교 * Feat: 카드 등록 로직 추가, 작업 진행중 * Feat: 초기 렌더링 구현 (#23) * Chore: eventDelegate함수 수정 for로직을 없애고 closest(selector) 존재하면 해당 원소를 this바인딩하여 핸들러를 호출하도록 변경 * Feat: webdb.js 추가 초기 데이터 추가 * Feat: webdb.js 함수 추가 getData: key값으로 로컬 스토리지 데이터 가져오기 getColumns: 모든 todoList 배열 가져오기 columns: todoList localStorage키 배열 * Remove: store.js 삭제 * Chore: devtool 옵션 추가 * Feat: TodoItem, TodoList 추가 * Chore: uuid 패키지 추가 * Chore: webdb 초기상태 업데이트 * Feat: Todo View 추가 * Refactor: index.html 수정 마크업 수정 * Feat: 접속시 렌더링 로직 구현, 카드 추가 구현중 * Refactor: View 이름 변경 (#24) TodoItem -> TodoCard TodoList -> TodoColumn * Style: lint 수정 (#25) * Refactor: 생성자호출할시 매겨변수 전달 (#26) 이전: - 생성자 호출 - init(data) 이후: - 생정자 호출(data) * Refactor: 돔 렌더링 로직 변경 (#28) * Refactor: 매개변수 전달 정리 constructor 내부에서 구조분해 했던 로직을 매개변수로 이동 * Refactor: 로직수정 - $todoColumn, $todoList 생성자에서 초기화 - 변수 네이밍 수정 * Refactor: 로직 변경 - columns -> todoColumns - todoColumns 구성 변경 * Feat: 카드 등록 기능 구현중 (#29) * Fix: 변수 오타 수정 line19: $todoList -> $todoColumn * Feat: setData 추가 * Feat: store 추가 * Fix: textarea의 width값 수정 * Rename: index.js 제거 임시로 생성했던 index.js 제거 * Feat: utils.js에 hasClass추가 클래스를 가지고 있는지 bool로 반환하여 알려주는 유틸 함수 추가 * Chore: eslint 설정 변경 class-method-use-this: this를 사용하지 않으면 스태틱 메서드로 전환하기 룰 비활성화 * Feat: Create Card 기능 진행중 title, desc가 둘 다 채워져 있어야만 등록버튼 활성화 구현 취소버튼 누르면 카드 삭제 구현 최대 1개까지만 등록폼 생성가능 구현 (전역에서 1개일지, 컬럼당 1개일지는 미정) * Feat: textarea auto resize, p <-> textarea 변환 (#31) * Design: todo-item의 title, desc 스타일 적용받는 셀렉터 추가 textarea -> item 등록, 수정시 p -> 그외 _todo-column.scss -> _todo-list.scss로 선언 이동 * Feat: replaceLFWithBrElement 추가 및 템플릿 변경 replaceLFWithBrElement - db에서 읽은 텍스트에서 개행문자를 <br>로 바꾸는 함수 추가 * Feat: replaceParagraphWithTextarea 추가 p엘리먼트를 textarea엘리먼트로 전환하는 함수 * Feat: textarea엘리먼트 autosize기능 추가 * Feat: text의 linebreak와 <br>을 교체하는 함수 추가 replaceLinebreakWithBrElement: \r, \r\n, \n를 <br>엘리먼트로 교체 * Feat: TodoCard에 static method 추가 TodoCard 엘리먼트의 title, desc를 나타내는 엘리먼트를 <p>에서 <textarea>로, 또는 반대로 바꾸는 메서드 추가 * Feat: Modal (#32) * Feat: modal 마크업 * Design: modal * Chore: modal default 디스플레이 hidden으로 변경 * Feat: 카드삭제버튼 누르면 모달창 보여주기 * Refactor: modal 마크업 수정 취소모달 -> 기본 모달 수정모달, 추가모달 등등 다른 내용으로 활용 가능하도록 * Feat: openModal, closeModal utils추가 및 등록 - app.js에서 closeModal 이벤트 위임 - openModal: modal의 컨텐츠를 넘겨 받을 수 있음. * Refactor: app.js에 init함수 추가 DOMContentLoaded이벤트로 init함수 실행 * Chore: modal 마크업 수정 scss: 공백 및 색깔 변수 수정 modal.js 삭제 * Feat: 카드 등록 추가 (#33) * Refactor: 중복 css선언 제거 * Refactor: Modal 분리 * Feat: 등록 버튼 이벤트핸들러 추가 * Feat: 카드 등록 추가 * Style: 메서드 호출 순서, 메서드 선언 순서 변경 (#34) Co-authored-by: YUNHO <[email protected]>

* feat : 테이블 인덱스 컬럼 추가 closes #15 * refactor: [BE] 1주차 금요일 pr 피드백 적용 closes #16 * feat : 로그로직 작성 - 로그 조회 기능 구현 - 로그 저장 기능 구현 closes #19 * feat: 카드 Status별로 조회 기능 구현 Closes #23 * feat: CardListDto 생성하고 관련 로직 수정 Closes #29 * refactor: 카드 생성 로직 변경 - addDto에 인덱스값과 스테이터스값 같이 받아서 생성 closes #31 * test: CardRepositoryTest 작성 * test: 테스트 작성하기 - 카드 컨트롤러와 리퍼지토리 테스트 하기 closes #21 * Build: build.gradle, application.properties 수정 - mysql 의존성 추가 - yml 파일 생성 Co-authored-by: leekm0310 <[email protected]>

[iOS] 네트워크 GET 요청 및 데이터 연결

* 카드 등록 - 새로운 카드 입력창 생성 기능 구현 (#36) * feature: 등록버튼을 누르면 새로운 카드, 카드 개수 렌더링 등록카드의 등록버튼으로 리스트에 새로운 카드를 추가하고 리스트의 카드 개수를 렌더링한다. * rename: todolistStore.js->todoListStore.js * feat: 새로운 카드 데이터 post 요청 등록버튼을 누르면 새로운 카드의 데이터를 post 요청하여 업데이트 * 카드 수정 - 내용 수정 기능 구현 (#37) * feat: 더블 클릭시 수정카드 전환, 수정버튼 활성화 롱클릭과 더블 클릭을 구분하고 수정카드에 기존과 다른 데이터가 입력되면 수정버튼을 활성화 * feat: 등록카드, 수정카드 데이터 fetch 요청 put 메소드로 변경된 카드 데이터를 서버에 보냄 * refactor: activationStore -> todoListStore 모델의 state를 todoListStore모듈에서 관리함 * 카드 삭제 - X 버튼 마우스 오버 이벤트 & 삭제 알럿창 구현 (#41) * design: delete alert 창 구현 * feat: task x delete button mouseover 이벤트 구현 * mouseover 시 task item 색 변화 * mouseout 시 원래대로 돌아옴 * feat: task title을 이용하여 task 를 삭제하는 함수 구현 * feat: 삭제 Alert 창 생성 기능 구현 * design: delete-button hover 이벤트 클래스로 분리 * feat: Alert 창 결과에 따른 Task 스타일 변화 기능 구현 * remove: todolistStore.js * rename: todoListStore.js * 카드 이동 - 리스트 중간에 카드 추가 기능 구현(1) (#42) * 카드 등록 - 새로운 카드 입력창 생성 기능 구현(1) (#30) * [team-19] Todo-list 1주차 #1 코드리뷰 요청 (#32) * Update issue templates * docs: Update Readme Readme 1차 업데이트 * docs: Update Readme 기술 스택 추가 * Design: reset scss 적용 * HTML, CSS 작업 - Header, Sidebar (#11) * design: header HTML markup Related to: #8 * design: header css style Related to: #8 * design: sidebar HTML markup Related to: #8 * design: sidebar css style Related to: #8 * 메뉴 - 우측 히든 레이어 생성, 애니메이션 구현 (#12) * feat: sidebar 메뉴 버튼 사이드바 메뉴 버튼 클릭시 사이드바 숨김,보임 기능 구현 Related to: #10 * design: sidebar 애니메이션 왼쪽방향으로 나타나고 오른쪽 방향으로 숨겨지는 애니메이션 구현 Related to: #10 * 메뉴 - 우측 히든 레이어 생성, 애니메이션 구현 (#13) * feat: sidebar 메뉴 버튼 사이드바 메뉴 버튼 클릭시 사이드바 숨김,보임 기능 구현 Related to: #10 * design: sidebar 애니메이션 왼쪽방향으로 나타나고 오른쪽 방향으로 숨겨지는 애니메이션 구현 Related to: #10 * feat: action분리 action에 따라 sidebar 리스트의 comment를 다르게 출력함 Related to: #10 * feat: timestamp 기능 구현 현재 시간을 기준으로 투두리스트 작성 및 변경 시간을 보여주는 timestamp 구현 Related to: #10 * feat: 을/를, 으로/로 구분 단어의 마지막 글자 종성에 따라 모음을 다르게 렌더하는 기능 구현 Related to: #10 * HTML, CSS 작업 - Card List (#14) * design: main column list html markup Related to : #9 * move: svg file move to public/svg * 기존 public 에 있던 svg 파일을 svg 폴더 내로 이동 * icon-add.svg 파일 추가 Related to : #9 * Design: font mixin 함수 추가 Related to : #9 * design: column list scss style 작성 Related to: #9 * design: column task list scss style 작성 Related to: #9 * design: task commnet padding 을 margin 으로 변경 Related to: #9 * design: sccs convert to css Related to: #9 * fix: icon svg 파일 경로 수정 Related to: #9 * chore: 빌드환경구성 (#16) babel, webpack 적용 Related to: #15 * 메뉴 - 사용자 액션 리스트 구현 (#19) * feat: sidebar 액션 리스트 히든레이어 영역을 애니메이션 효과와 함께 숨김,보임 기능 구현 Related to: #17 * design: sidebar 액션리스트 애니메이션 Related to: #17 * move: package.json, gitignore * feat: sidebar 스크롤 기능 구현 액션리스트 기록이 많아지면 스크롤을 이용하여 다음 기록을 확인할 수 있음 Related to: #17 * design: 액션리스트 스크롤 구현 Related to: #17 * feature: json server 생성 및 db.json 추가 (#21) * json server 설치하고 db.json 에 필요한 데이터들을 추가함 Related to: #18 * HTML, CSS 작업 - Card (#22) * feat: todo-list 입력창 높이 자동 조절 todo-list를 입력하거나 지울 때 입력창의 높이를 자동으로 조절하는 기능 구현 Related to: #20 * design: column button hover color 칼럼 타이틀의 +,x 버튼을 hover할 때 버튼 색상 변경 Related to: #20 * design: 수정카드 디자인 구현 Related to: #20 * 리팩토링 - 1주차 리뷰 반영 (#24) * refactor: getFinalConsonant 매직넘버 제거 Related to: #23 * refactor: calcTimeForToday 함수 리팩토링 * 매직 넘버 제거 * DayDifference 식 수정 * 몇 년전 조건 추가 Related to: #23 * refactor: writeTime -> timeStamp Related to: #23 * refactor: identifyCategory 함수 리팩토링 * 구조 분해 할당 적용 Related to: #23 * refactor: svg 파일 경로 변수로 분리 * svg 파일 경로를 constants의 imagePath에 변수로 생성해서 관리하도록 변경 Related to: #23 * feat: fetchData 함수 구현 Related to: #23 * chore: bundle.js Related to: #23 * refactor: sidebar model 리팩토링 Related to: #23 * refactor: activationStore import 방식 변경 Related to: #23 Co-authored-by: Hemdi <[email protected]> Co-authored-by: Hemudi <[email protected]> * 카드 등록 - 새로운 카드 등록 기능 구현 (#29) * feat: todolist column과 task를 스크립트로 렌더링 list view와 task view를 클래스로 생성하고 db.json의 todolist를 하나의 배열로 묶음 * feat: column add button 칼럼의 추가버튼으로 등록카드 생성,삭제 기능 구현 * feat: task cancle button 등록카드의 취소버튼을 누르면 등록카드를 삭제하는 기능 구현 * refactor: 메인 렌더링 영역을 list에서 main으로 수정 entry point에서 mainInit의 parent를 list에서 main으로 리팩토링 * refactor: input event를 메인에서 task 모듈로 분리 입력창을 autoResize하는 이벤트를 task 모듈로 분리함 * rename: index.js * feat: 등록버튼 활성화 기능 구현 등록카드에 내용을 입력하면 등록버튼을 활성화, 입력할 수 있는 최대 글자수는 500자 * refactor: registration activation을 todoListStore에서 관리 activiation관리를 task에서 todoListStore로 변경 Co-authored-by: Hemdi <[email protected]> Co-authored-by: Hemudi <[email protected]> * Revert "카드 등록 - 새로운 카드 입력창 생성 기능 구현(1) (#30)" (#31) This reverts commit 7155f85. * feat: 칼럼 사이 카드 이동 마우스 포인터를 기준으로 드래그중인 카드 아래 새로운 카드 존재 여부에 따라 해당 카드 앞 또는 가장 아래에 드래그 카드 추가 * rename: deleteMenu.js->alertDelete.js * feat: 드래그앤 드랍 적용 task 인스턴스에 드래그앤드랍 기능 적용 * refactor: export 방식 변경 Co-authored-by: Hemdi <[email protected]> Co-authored-by: Hemudi <[email protected]> * feat: task id 추가 + id를 이용한 삭제 기능 구현 (#44) * 기존 task title 을 이용해 삭제했던 기능을 id 를 이용해 삭제하는 방식으로 변경 Co-authored-by: Hemdi <[email protected]> Co-authored-by: Hemudi <[email protected]>

- CardDTO에 Comparable을 구현. - ClassifiedCardsDTO의 sort메서드를 통해 정렬. Related to #16

테스트 코드 작성- 테스트 실패 뜸(디버깅 필요) 타입, URL 을 인자로 받아서 클래스 초기화 하도록 변경 테스트 가능한 클래스로 수정 API 호출시 인자값을 적게 받아 사용할 수 있는점에 중점을 둠

Result 의 failure 가 겹치므로 notConnect, notConvert 로 수정

fetch 에 필요한 함수만 상위 클래스에서 가져와 상속 관계를 줄임 서버에서 가져오는 데이터인 Cards 구조체 선언

[Refactor] CardsFetchingDataTask 리펙토링 (#23)

[funny] 목록 삭제시 스크롤 올라가는 문제 수정, 스와이프 삭제 구현 중

leekm0310 and others added 11 commits April 5, 2022 12:06

Update README.md

9d7e0ea

Update issue templates

742541c

Merge pull request #1 from KTH-96/KTH-96-patch-1

694a1e4

Update issue templates

refeat : 프로젝트 구조 개선

7600317

build : jdbc와 h2의 의존성 추가

f25799b

closes #5

feat : 공통 패키지 구조 작성

b346450

- domain, service, web 구조로 작성 close #5

Merge branch 'feature_#5_BE_common1' into dev-BE

dcc5821

feat : 카드 삭제 기능 구현

3750280

closes #7

Merge branch 'feature_#7' into dev-BE

4109e99

feat : 카드 저장 로직 구현

654ecd2

close #6

conflict 해결

564fc40

KTH-96 added the review-BE Improvements or additions to documentation label Apr 6, 2022

KTH-96 requested a review from Min-92 April 6, 2022 07:55

Min-92 approved these changes Apr 6, 2022

View reviewed changes

Min-92 merged commit e375db2 into codesquad-members-2022:team-31 Apr 6, 2022

Hyune-c pushed a commit that referenced this pull request Apr 6, 2022

feat: h2 DB 설정과 TODO 테이블 생성 (#23)

82ebf68

- application.yml에 h2 관련 설정 - TODO 테이블과 더미 데이터 생성 (schema.sql, data.sql) closes #21

guswns1659 pushed a commit that referenced this pull request Apr 6, 2022

🔀 Merge pull request #23 from who-hoo/Hoo#009

259811a

[후] 카드(할일) 도메인 개발 리팩토링

GangWoon pushed a commit that referenced this pull request Apr 10, 2022

Merge pull request #23 from zbqmgldjfh/feature-view-cardTableView

fb1b0a9

[iOS] CardTableView와 CardCell 구현

wooody92 pushed a commit that referenced this pull request Apr 10, 2022

feat: [#23] Card 도메인 객체 추가 및 관련 레이어 클래스 추가

518425a

wooody92 pushed a commit that referenced this pull request Apr 10, 2022

feat: [#23] Card 레이어 통합/단위 테스트 클래스 추가

462aa62

wooody92 pushed a commit that referenced this pull request Apr 10, 2022

Merge pull request #27 from rxdcxdrnine/feature/#23-card_boilerplate

647d3e2

[BE] 도메인 객체 추가 및 레이어 클래스 추가

GangWoon pushed a commit that referenced this pull request Apr 10, 2022

Merge pull request #23 from jeremy0405/feature/iOS/Navigation

f8e57f1

MainVC와 InspectorVC를 관리하는 ContainerVC 생성

piggyse pushed a commit that referenced this pull request Apr 11, 2022

[Feat][#23]활동기록뷰 만들기

865968d

Content - 뷰 애니메이션 기능 구현 Todo - 활동기록 테이블뷰 구현

fivethreeeo pushed a commit that referenced this pull request Apr 13, 2022

Design: 남은 정적페이지 분량 (#23)

12be595

* Design: 카드등록창 이미지 추가 * Design: Card 등록박스 임시로 추가 * Refactor: textarea rows 속성 추가

piggyse pushed a commit that referenced this pull request Apr 13, 2022

[Refactor][#23]활동기록뷰 레이아웃 수정

1998d1f

Content - 기존 레이아웃 수정

piggyse pushed a commit that referenced this pull request Apr 13, 2022

[Refactor][#23]활동기록 뷰 레이아웃 수정

3a56ea0

Content - 어색한 애니메이션 수정 Todo - 활동기록 테이블뷰,셀 만들기

piggyse pushed a commit that referenced this pull request Apr 13, 2022

[Refactor][#23]활동기록뷰 레이아웃 수정

0c3005e

Content - 활동 기록 뷰 레이아웃 수정

snowjang24 pushed a commit that referenced this pull request Apr 13, 2022

[#23]Feat: json-server에서 column title을 가져오는 api 함수 추가

2597454

snowjang24 pushed a commit that referenced this pull request Apr 13, 2022

[#23]Feat: 하드코딩한 columns 데이터 서버 데이터로 변경

f932415

lenaios pushed a commit that referenced this pull request Apr 14, 2022

Merge pull request #23 from wnsxor1993/feature/connectModelVC

28f7bba

[iOS] 네트워크 GET 요청 및 데이터 연결

tmdgusya pushed a commit that referenced this pull request Apr 14, 2022

[BE-pio] feat: 전체조회시 카드를 order순으로 정렬 (#23)

49652f1

- CardDTO에 Comparable을 구현. - ClassifiedCardsDTO의 sort메서드를 통해 정렬. Related to #16

SangHwi-Back pushed a commit that referenced this pull request Apr 15, 2022

refactor: DataTaskError 의 case 네이밍 수정 (#23)

b5a94a6

Result 의 failure 가 겹치므로 notConnect, notConvert 로 수정

SangHwi-Back pushed a commit that referenced this pull request Apr 15, 2022

Merge pull request #38 from Jinsujin/feature-23

8c707bb

[Refactor] CardsFetchingDataTask 리펙토링 (#23)

astraum pushed a commit that referenced this pull request Apr 15, 2022

Merge pull request #23 from ese111/main

d2082ae

[funny] 목록 삭제시 스크롤 올라가는 문제 수정, 스와이프 삭제 구현 중

		private final static String DELETE_CARD_SQL = "delete from card where id = ?";
		private final static String INSERT_CARD_SQL = "insert into card(title, contents, writer, createTime, status) value (?,?,?,?,?)";

		@@ -0,0 +1,11 @@
		package com.todolist.project.domain;

		public enum Status {

		compileOnly 'org.projectlombok:lombok'
		annotationProcessor 'org.projectlombok:lombok'

[team-31][BE][Yan-K] To-do list 그룹 프로젝트 1주차(수) PR #23

[team-31][BE][Yan-K] To-do list 그룹 프로젝트 1주차(수) PR #23

Uh oh!

Conversation

KTH-96 commented Apr 6, 2022

구현사항

어려웠던 부분

Uh oh!

Min-92 left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Min-92 left a comment •

edited

Loading