Skip to content
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

feat : complete progress page #88

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

SamTheKorean
Copy link
Contributor

@SamTheKorean SamTheKorean commented Nov 23, 2024

체크리스트

  • 이슈가 연결되어 있나요?
  • 배포 후 브라우저 콘솔에 경고나 오류가 있나요?

@SamTheKorean SamTheKorean self-assigned this Nov 23, 2024
@SamTheKorean SamTheKorean linked an issue Nov 23, 2024 that may be closed by this pull request
@SamTheKorean SamTheKorean changed the title Complete progress page feat : complete progress page Nov 23, 2024
@SamTheKorean
Copy link
Contributor Author

SamTheKorean commented Nov 23, 2024

Screenshot 2024-11-23 at 3 59 32 PM
디테일한 ui나 데이터를 가공해줘야하는데 데모를 위해 해당 테스크들은 뒤로 미루고 지금까지 나온 부분까지 공유드립니다! 내부에 있는 링크들 포함 문제들 전부 실제 데이터와 연결했습니다!

@DaleSeo
Copy link
Contributor

DaleSeo commented Nov 23, 2024

디테일한 ui나 데이터를 가공해줘야하는데 데모를 위해 해당 테스크들은 뒤로 미루고 지금까지 나온 부분까지 공유드립니다! 내부에 있는 링크들 포함 문제들 전부 실제 데이터와 연결했습니다!

@SamTheKorean 프로그레스 페이지까지 데모를 하기에는 너무 PR이 늦게 올라온 거 아닌가 싶어요 😓 너무 서두르다가 다른 부작용을 일으키기 보다는 우리가 비현실적인 목표를 설정했다는 것을 인정하고, 너무 무리하지 말고 오늘은 아쉽지만 되는데까지 데모를 하면 어떨까요?

@@ -0,0 +1,92 @@
import { useEffect } from "react";
import styles from "./Aside.module.css";
import Seed from "../../assets/Seed.png";
Copy link
Contributor

Choose a reason for hiding this comment

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

assets 디렉토리에 이미지 파일이 없는 것 같은데요?

Shot 2024-11-23 at 16 29 24

Copy link
Contributor Author

@SamTheKorean SamTheKorean Nov 23, 2024

Choose a reason for hiding this comment

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

아 드래프트로 올려놓고 다른 pr 마지되면 rebase해서 가져와서 올리고 pr 오픈할 예정이었습니다..! 여기에 올리면 충돌을 해결해야할 것 같아서요.! 생각해보니 같은 파일이라 충돌은 안날 수 있겠네요..! 떠나서 다른 분들에게 혼란을 드리는 것 같으니 이제는 충돌이 예상되더라도 우선 올리고 해결하겠습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Entire Progress Page
2 participants