Feat: 내 일정 페이지 추가 #30

pepperdad · 2023-11-03T07:37:42Z

구현사항

일정만들기 1 페이지

ScheduleTab, Tab 컴포넌트 추가
ScheduleCard 컴포넌트 추가
필터링 기능 추가

변경사항

일정만들기 1 페이지 -> 모든 일정 페이지로 변경
내 일정 페이지 추가

기존에 구현한 일정만들기 1 페이지를 모든 일정 페이지로 경로 수정
내 일정 페이지 기본 UI 추가
필터링 로직 수정 : 추후 로직 수정 필요

테스트

localhost:3000/schedule/ 접근 (내 일정 페이지)
localhost:3000/schedule/all 접근 (모든 일정 페이지)

TODO

탭 전환시 쿼리를 이용해서 동적으로 데이터 불러올지, 혹은 현재 있는 데이터에서 필터링 할지 고민할 것 -> 현재는 프론트에서 데이터 필터링
삭제하기 버튼 클릭시, 서버에 삭제 요청 추가
내가 만든 일정 필터링 추가

Choi-Jinwook

확인했습니다! 코멘트 확인부탁드릴게요😀

Choi-Jinwook · 2023-11-03T08:45:53Z

src/create/components/ScheduleCard.tsx

+          {status ? (
+            <>
+              <div className="w-[7px] h-[7px] bg-red-600 rounded-full" />
+              <div className="ml-2 text-xs text-neutral-400">완료</div>
+            </>
+          ) : (
+            <>
+              <div className="w-[7px] h-[7px] bg-amber-300 rounded-full" />
+              <div className="ml-2 text-xs text-neutral-400">작성 중</div>
+            </>
+          )}


[Lv1] 이 부분 일정 구분이 완료, 작성 중 말고 더 있을 것 같은데 일단 임시로 해놓으신 걸까요??
기획/디자인쪽이랑 이야기해보고 컴포넌트로 관리하는것이 좋을 것 같습니다!

넵 맞습니다! 이 부분은 기획/디자인 부분이랑 얘기가 필요할 것 같네요!!

into feat/create-schedule ; Conflicts: ; tsconfig.json

tsconfig.json

NacreousCloud

고생하셨어요~ 코멘트 확인 부탁드립니다~

src/create/components/ScheduleTab.tsx

NacreousCloud · 2023-11-05T09:19:49Z

src/create/components/Create.tsx

+    }
+  };
+
+  // date, tab, title 이 바뀔 때마다 필터링 진행


[Lv3] 바뀔때마다 필터링하는 기획이 있나요?? 제가 확인해봤을땐 안보였는데, 해당 이펙트는 리소스 소모가 클것같아서 조정이 필요해보입니다. 기획팀이나 디자인팀에 문의해보죠!

이 부분은 제가 기획쪽 체크를 못했던 것 같습니다! 기존 디자인 화면에서는 찾기 버튼이 없어서, 바뀔 때 마다 필터링하는 로직으로 확인하고 구현했는데 지금 확인해보니 버튼이 생겼네요 ㅎㅎ
필터링말고 찾기버튼을 누르면 쿼리를 날리는 방식으로 수정하면 될 것 같습니다! 감사합니다 ㅎㅎ

NacreousCloud · 2023-11-05T09:58:29Z

src/create/components/InputCalender.tsx

+        <div className="absolute top-[38px] left-0 z-30">
+          <Calendar
+            value={date || undefined}
+            onChange={(newDate: any) => handleDateChange(newDate)}


Suggested change

onChange={(newDate: any) => handleDateChange(newDate)}

onClickDay={handleDateChange}

[Lv3] Day 클릭시 발생하는 이벤트로 대체하면 any 타입 명시 없이 사용가능합니다.
타입단언만큼, any 타입의 사용도 최소한으로 사용하는것이 좋습니다!

확인했습니다 :)

src/create/components/Create.tsx

into feat/create-schedule ; Conflicts: ; tsconfig.json

.pnp.cjs

package.json

tsconfig.json

yarn.lock

pepperdad · 2023-11-08T12:52:44Z

다시 확인 부탁드립니다!
Review 남겨주신 것들 수정 및 기존의 일정 만들기 1 페이지 -> 모든 일정 페이지로 수정하고, 내 일정 페이지의 헤더와 탭 추가하였습니다.
PR이 조금 길어져서 Approve 시, merge 후 다시 기능별 브랜치 파서 작업 후 PR 올리겠습니다.

NacreousCloud · 2023-11-16T16:36:41Z

src/schedule/components/ScheduleTab.tsx

+        {tabTitle}
+      </div>
+      <div className="flex mt-9 gap-x-10">
+        {tabItems.map((tab, i) => (


Suggested change

{tabItems.map((tab, i) => (

{tabItems.map(({title}, i) => (

가독성을 위한 취향차이긴하지만 map 매개변수를 구조분해로 받는다면 아래 코드들에서 tab.title을 title로만 사용할수 있습니다!

NacreousCloud · 2023-11-16T17:17:55Z

src/schedule/components/All.tsx

+    };
+
+    console.log("searchData", searchData);
+    // TODO: API 요청 추가 ??? 아님 프론트에서 다시 필터 ??? <- 확인해야 함


보통 검색은 프론트 필터링 없이 api 요청으로만 이뤄집니다!

NacreousCloud · 2023-11-16T17:20:46Z

src/schedule/components/All.tsx

+  };
+
+  // tab으로 필터링
+  const filteredInTab = (tab: string, dataList: CardItemType[]) => {


위 답변이 해당 함수에도 적용됩니다! api 요청을 위해 onClickSearch 내부 searchData 에 status 등을 추가하는 방식으로 추후 수정되어야할거같아요! 지금 당장은 작업하지않아도 되니 참고만 해주세요!

NacreousCloud · 2023-11-16T17:23:08Z

src/schedule/components/AllContent.tsx

+  const [isMySchedule, setIsMySchedule] = useState(false);
+
+  // TODO: 내가 만든 일정만 필터 로직 추가
+  const onClickMySchedule = () => setIsMySchedule((prev) => !prev);


내가 만든 일정만 체크하는것도 비슷할겁니다! 필터링하는 로직은 검색 api 쏠때 파라미터로만 추가하면될것같아요!

NacreousCloud

확인했습니다! 추가사항이나 따로 이상이 없다면 머지 해주세요~ 고생하셨습니다!

into feat/create-schedule ; Conflicts: ; .pnp.cjs ; yarn.lock

github-actions · 2023-11-20T16:09:06Z

.pnp.cjs

+          ],\
+          "linkType": "HARD"\
+        }]\
+      ]],\
      ["titleize", [\
        ["npm:3.0.0", {\
          "packageLocation": "./.yarn/cache/titleize-npm-3.0.0-7deac2f3a3-71fbbeabbf.zip/node_modules/titleize/",\


The current Secretary-General of the United Nations is António Guterres.

github-actions · 2023-11-20T16:09:34Z

package.json

@@ -18,6 +18,7 @@
    "lodash": "^4.17.21",
    "next": "12.3.2",
    "react": "18.2.0",
+    "react-calendar": "^4.6.1",
    "react-dom": "18.2.0",
    "react-query": "^3.39.3",
    "recoil": "^0.7.7",


이 코드 패치를 간단히 검토해 드리겠습니다.

버그 위험:

이 코드 패치에는 특별한 버그 위험이 보이지 않습니다.

개선 제안:

패치에서 react-calendar 패키지를 추가하고 있습니다. 하지만 다른 종속성들의 버전은 수정되지 않고 있습니다. 종속성 버전 충돌 등의 문제가 발생하지 않는지 확인해야 합니다.

보안 및 성능 측면에서, 종속성들을 최신 버전으로 업데이트하는 것도 고려할 가치가 있습니다. 새로운 버전에서는 여러 가지 개선 사항과 버그 수정이 포함되어 있을 수 있습니다.

github-actions · 2023-11-20T16:20:29Z

.pnp.cjs

+          ],\
+          "linkType": "HARD"\
+        }]\
+      ]],\
      ["titleize", [\
        ["npm:3.0.0", {\
          "packageLocation": "./.yarn/cache/titleize-npm-3.0.0-7deac2f3a3-71fbbeabbf.zip/node_modules/titleize/",\


As an AI language model, I don't have real-time data or access to current events beyond my last update in September 2021. Please verify any information I provide with up-to-date sources.

github-actions · 2023-11-20T16:21:05Z

yarn.lock

+  checksum: da62c4acac565902f0624b123eed6dd3509bc9a8d30c06e017104bedcf5d35810da8ff72864400ad19c5c7806fc0a8323c68baf3e326af7cb7d969f846100d71
+  languageName: node
+  linkType: hard
+
 "titleize@npm:^3.0.0":
  version: 3.0.0
  resolution: "titleize@npm:3.0.0"


아래는 코드 패치입니다. 이를 간단히 검토해 드리겠습니다. 버그 위험과/또는 개선 제안이 있으면 언제든지 알려주세요:

"@types/lodash.memoize" 패키지 버전을 4.1.8로 업데이트하는 것이 좋을 것 같습니다.

"@types/lodash@npm:^4.14.200" 패키지의 올바른 버전 범위를 명시해야 합니다.

새로운 패키지인 "@wojtekmaj/date-utils@npm:^1.1.3"와 "@xmldom/xmldom@npm:^0.8.3"을 추가할 수 있습니다.

"react-calendar" 패키지를 "^4.6.1" 버전으로 추가하고 필요한 종속성을 설치하는 것이 좋습니다.

위 내용들을 고려하여 코드를 수정하면 개선될 것입니다.

github-actions · 2023-11-20T16:25:27Z

yarn.lock

+  checksum: da62c4acac565902f0624b123eed6dd3509bc9a8d30c06e017104bedcf5d35810da8ff72864400ad19c5c7806fc0a8323c68baf3e326af7cb7d969f846100d71
+  languageName: node
+  linkType: hard
+
 "titleize@npm:^3.0.0":
  version: 3.0.0
  resolution: "titleize@npm:3.0.0"


해당 코드 패치의 리뷰를 도와드리겠습니다. 가능한 간결하게 답변하도록 하겠습니다.

@types/lodash 패키지 버전이 업데이트되었습니다.

@types/lodash.memoize 패키지가 추가되었고, 이에 대한 종속성인 @types/lodash가 필요합니다.

@wojtekmaj/date-utils, clsx, get-user-locale, prop-types, tiny-warning 패키지가 추가되었고, 각각에 대한 종속성과 피어 종속성이 정의되어 있습니다.

react-calendar 패키지 버전이 업데이트되었고, 이에 대한 종속성인 @wojtekmaj/date-utils, clsx, get-user-locale, prop-types, tiny-warning 패키지가 필요합니다.

lodash.memoize 패키지 버전이 업데이트되었습니다.

prop-types 패키지 버전이 업데이트되었습니다.

개선 제안:

종속성 버전 관리를 더 정확하게 할 수 있도록 패키지 버전을 명시적으로 지정하는 것이 좋습니다.

비슷한 종류의 패키지들을 그룹화하거나, 의존성 관리를 좀 더 명확하게 표현할 수 있는 방법을 고려해 볼 수 있습니다.

버그 위험:

코드 패치에서 특정한 버그 리스크는 발견되지 않았습니다.

주의 사항:

코드 리뷰에 제시된 내용만으로 해당 코드의 전반적인 상황을 파악하기는 어렵기 때문에, 추가적인 정보가 필요할 수도 있습니다.

NacreousCloud

ACK

pepperdad added 5 commits November 2, 2023 01:04

feat: 일정만들기 기본 UI 추가

323e462

design: 기본 asset 추가

13f3def

feat: Tab 컴포넌트 추가

8b8cf5a

feat: 일정 카드 컴포넌트 추가

8788efb

feat: 일정만들기 1 페이지 추가

3e9a249

pepperdad requested review from NacreousCloud and Choi-Jinwook November 3, 2023 07:38

Choi-Jinwook reviewed Nov 3, 2023

View reviewed changes

pepperdad added 5 commits November 4, 2023 16:39

chore: react-calendar 라이브러리 설치

0ede09c

feat: InputCalendar 컴포넌트 추가

3f939b5

feat: 제목, 기간, 탭 별 필터링 기능 추가

dc2ff27

feat: Tab 명칭을 ScheduleTab으로 변경

77fb2dd

feat: 추가 탭 구분 컴포넌트 추가

1710647

pepperdad requested a review from Choi-Jinwook November 4, 2023 13:42

pepperdad added 2 commits November 4, 2023 22:44

Merge branch 'develop' of https://github.com/Project-Catcher/catcher-web

1f52232

into feat/create-schedule ; Conflicts: ; tsconfig.json

feat: merge commit

89bc522

github-actions bot reviewed Nov 4, 2023

View reviewed changes

tsconfig.json Outdated Show resolved Hide resolved

NacreousCloud requested changes Nov 5, 2023

View reviewed changes

pepperdad added 12 commits November 7, 2023 18:16

feat: 타입단언 로직 수정

6aaf42a

feat: 중복되는 조건 최상단으로 올림

e851d8f

feat: 찾기 버튼 추가

6272c4c

feat: 내가 만든 일정 버튼 추가

32d90ac

feat: 찾기 버튼 클릭시, 데이터 요청 로직 추가

960ac97

design: asset 추가

bad2e94

feat: ScheduleCard UI 수정

b243dee

fix: 내 일정, 모든 일정 페이지 경로 수정

2f5d0c9

feat: 일정 탭 컴포넌트 일정 페이지에서 공용으로 사용하도록 변경

281e41f

feat: 일정 페이지 헤더 탭 수정

dbda2d1

feat: 내 일정 페이지 기본 UI 추가

189501c

Merge branch 'develop' of https://github.com/Project-Catcher/catcher-web

e0fe189

into feat/create-schedule ; Conflicts: ; tsconfig.json

github-actions bot reviewed Nov 8, 2023

View reviewed changes

.pnp.cjs Outdated Show resolved Hide resolved

github-actions bot reviewed Nov 8, 2023

View reviewed changes

package.json Show resolved Hide resolved

github-actions bot reviewed Nov 8, 2023

View reviewed changes

tsconfig.json Show resolved Hide resolved

github-actions bot reviewed Nov 8, 2023

View reviewed changes

yarn.lock Show resolved Hide resolved

pepperdad requested a review from NacreousCloud November 8, 2023 12:50

pepperdad changed the title ~~Feat: 일정만들기 1 페이지 추가~~ Feat: 내 일정 페이지 Nov 8, 2023

pepperdad changed the title ~~Feat: 내 일정 페이지~~ Feat: 내 일정 페이지 추가 Nov 8, 2023

feat: AllContent 컴포넌트 추가

54e83a1

NacreousCloud reviewed Nov 16, 2023

View reviewed changes

NacreousCloud approved these changes Nov 16, 2023

View reviewed changes

Merge branch 'develop' of https://github.com/Project-Catcher/catcher-web

cbd0d26

into feat/create-schedule ; Conflicts: ; .pnp.cjs ; yarn.lock

github-actions bot reviewed Nov 20, 2023

View reviewed changes

fix: 충돌표식 제거

21ce528

pepperdad requested a review from NacreousCloud November 20, 2023 16:20

github-actions bot reviewed Nov 20, 2023

View reviewed changes

pepperdad added 2 commits November 21, 2023 01:23

fix: error resolve

2d19973

fix: error resolve

94512ef

github-actions bot reviewed Nov 20, 2023

View reviewed changes

NacreousCloud approved these changes Nov 20, 2023

View reviewed changes

pepperdad merged commit c70c8ca into develop Nov 20, 2023
2 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feat: 내 일정 페이지 추가 #30

Feat: 내 일정 페이지 추가 #30

pepperdad commented Nov 3, 2023 •

edited

Loading

Choi-Jinwook left a comment

Choi-Jinwook Nov 3, 2023

pepperdad Nov 4, 2023

NacreousCloud left a comment

NacreousCloud Nov 5, 2023

pepperdad Nov 7, 2023 •

edited

Loading

NacreousCloud Nov 5, 2023

pepperdad Nov 7, 2023

pepperdad commented Nov 8, 2023

NacreousCloud Nov 16, 2023

NacreousCloud Nov 16, 2023

NacreousCloud Nov 16, 2023

NacreousCloud Nov 16, 2023

NacreousCloud left a comment

github-actions bot Nov 20, 2023

github-actions bot Nov 20, 2023

github-actions bot Nov 20, 2023

github-actions bot Nov 20, 2023

github-actions bot Nov 20, 2023

NacreousCloud left a comment

	onChange={(newDate: any) => handleDateChange(newDate)}
	onClickDay={handleDateChange}

Feat: 내 일정 페이지 추가 #30

Feat: 내 일정 페이지 추가 #30

Conversation

pepperdad commented Nov 3, 2023 • edited Loading

구현사항

변경사항

테스트

TODO

Choi-Jinwook left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

NacreousCloud left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

pepperdad Nov 7, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

pepperdad commented Nov 8, 2023

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

NacreousCloud left a comment

Choose a reason for hiding this comment

github-actions bot Nov 20, 2023

Choose a reason for hiding this comment

github-actions bot Nov 20, 2023

Choose a reason for hiding this comment

github-actions bot Nov 20, 2023

Choose a reason for hiding this comment

github-actions bot Nov 20, 2023

Choose a reason for hiding this comment

github-actions bot Nov 20, 2023

Choose a reason for hiding this comment

NacreousCloud left a comment

Choose a reason for hiding this comment

pepperdad commented Nov 3, 2023 •

edited

Loading

pepperdad Nov 7, 2023 •

edited

Loading