Skip to content

[Refactor] 피드백 반영 및 대시보드 컬럼 및 카드 리스트 리팩토링#160

Closed
cccwon2 wants to merge 22 commits intodevelopfrom
김원

Hidden character warning

The head ref may contain hidden characters: "\uae40\uc6d0"
Closed

[Refactor] 피드백 반영 및 대시보드 컬럼 및 카드 리스트 리팩토링#160
cccwon2 wants to merge 22 commits intodevelopfrom
김원

Conversation

@cccwon2
Copy link
Copy Markdown
Contributor

@cccwon2 cccwon2 commented Nov 2, 2024

대시보드 컬럼별 카드 리스트 무한 스크롤과 드래그 앤 드랍을 동시 지원(창기님과 협업)
Issue #158
Issue #161

로그인 하면 내 대시보드에서 무한 로딩이 발생하는 버그
Issue #162

대시보드 삭제 안되는 버그
Issue #164

할 일 수정에서 이전 대시보드 상태 보이는 버그
Issue #155

특정 컬럼에 카드가 1개 있을 때 카드 수정 모달로 상태 변경을 하면 이전 자리에 카드가 그대로 남아있는 버그
Issue #163

@cccwon2 cccwon2 requested review from ChungO5, byeolee1221, hongggyelim and hyemeeny and removed request for hongggyelim November 2, 2024 08:54
@cccwon2 cccwon2 self-assigned this Nov 2, 2024
@cccwon2 cccwon2 added the 📦️ Chore 그 외 자잘한 수정에 대한 커밋 label Nov 2, 2024
@netlify
Copy link
Copy Markdown

netlify bot commented Nov 2, 2024

Deploy Preview for taskify-test-9-4 ready!

Name Link
🔨 Latest commit e5b40de
🔍 Latest deploy log https://app.netlify.com/sites/taskify-test-9-4/deploys/67282cf4ecaf1c0008a889db
😎 Deploy Preview https://deploy-preview-160--taskify-test-9-4.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@cccwon2 cccwon2 requested a review from hongggyelim November 2, 2024 10:15
@cccwon2 cccwon2 changed the title [Chore] 충오님, 혜민님 피드백 반영 [Chore] 충오님, 혜민님 피드백 반영 및 대시보드 컬럼 및 카드 리스트 리팩토링 Nov 2, 2024
@cccwon2 cccwon2 changed the title [Chore] 충오님, 혜민님 피드백 반영 및 대시보드 컬럼 및 카드 리스트 리팩토링 [Refactor] 피드백 반영 및 대시보드 컬럼 및 카드 리스트 리팩토링 Nov 2, 2024
@cccwon2 cccwon2 added 🔨 Refactor 코드 리팩토링에 대한 커밋 and removed 📦️ Chore 그 외 자잘한 수정에 대한 커밋 labels Nov 2, 2024
@netlify
Copy link
Copy Markdown

netlify bot commented Nov 3, 2024

Deploy Preview for taskify-codeit-94 ready!

Name Link
🔨 Latest commit e5b40de
🔍 Latest deploy log https://app.netlify.com/sites/taskify-codeit-94/deploys/67282cf4b01dfc0008a15c0f
😎 Deploy Preview https://deploy-preview-160--taskify-codeit-94.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

if (isDashboardInfoLoading || isMemberLoading) return null;

// 실제 표시할 대시보드 정보
const displayDashboard = dashboardInfo;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

별도의 변수명으로 가져와야하는 이유가 있을까요?

</label>
<div className="relative flex w-full flex-col space-y-[2px] bg-white" ref={dropdownRef}>
<div className="flex h-[50px] w-full items-center justify-between overflow-hidden rounded-lg px-4 ring-1 ring-inset ring-gray03 transition-all focus-within:ring-violet01 focus:outline-none focus:ring-1 focus:ring-inset">
<div className="flex h-[50px] w-[217px] items-center justify-between overflow-hidden rounded-lg px-4 ring-1 ring-inset ring-gray03 transition-all focus-within:ring-violet01 focus:outline-none focus:ring-1 focus:ring-inset">
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

이부분은 input 크기 변경하기로 한건가요?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

네 시안과 다르게 검색하면 해당 드롭다운이 작아져서 고정시켰습니다.

{isLoading && (
<div className="py-2 text-center">
<p className="font-bold text-gray01">카드 불러오는 중...</p>
</div>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

image
로딩 표시가 두 개가 보여서 이부분이 조금 텍스트가 너무 많은것같다는 느낌을 받았습니다

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

반영하겠습니다!

Copy link
Copy Markdown
Collaborator

@hongggyelim hongggyelim left a comment

Choose a reason for hiding this comment

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

수고하셨습니다

@cccwon2 cccwon2 closed this Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔨 Refactor 코드 리팩토링에 대한 커밋

Projects

Status: 완료

Development

Successfully merging this pull request may close these issues.

3 participants