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

2025 봄 이벤트: 스켈레톤 코드 #849

Merged
merged 4 commits into from
Feb 11, 2025

Conversation

malloc3141
Copy link
Contributor

Summary

It closes #848
2024fall 이벤트 코드를 복사하여 스켈레톤으로 활용할 수 있게끔 했습니다.

Images or Screenshots

Further Work

  • Do something...

@malloc3141 malloc3141 self-assigned this Feb 7, 2025
@malloc3141 malloc3141 requested review from kmc7468, jinhyeonkwon and Hyogyeong8 and removed request for kmc7468 and jinhyeonkwon February 7, 2025 13:53
Copy link

netlify bot commented Feb 7, 2025

Deploy Preview for taxi-dev-preview ready!

Name Link
🔨 Latest commit 8cc3094
🔍 Latest deploy log https://app.netlify.com/sites/taxi-dev-preview/deploys/67a895beb91b540008b60e5b
😎 Deploy Preview https://deploy-preview-849--taxi-dev-preview.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.

Comment on lines 17 to 21
const ModalEvent2025SpringShare = ({
isOpen,
onChangeIsOpen,
inviteUrl,
}: ModalEvent2025SpringShareProps) => {
Copy link
Member

Choose a reason for hiding this comment

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

엇 정렬이 좀 특이한데 호옥시 prettier 적용이 되어 있는걸까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

앗 설정이 제대로 안 되어 있었어요 수정했습니다!

: event2025SpringInfo.isAgreeOnTermsOfEvent === false
? [true, "이벤트에 참여해야 합니다"]
: event2025SpringInfo.creditAmount < bettingAmount
? [true, "송편코인이 부족합니다"]
Copy link
Contributor

@jinhyeonkwon jinhyeonkwon Feb 8, 2025

Choose a reason for hiding this comment

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

[나중에 쓸 정보]
이번 봄 이벤트 코인 이름을 안 정했었죠 아마..? 코인 이름이 정해지면 추후 PR에서 잊지말고 수정해 주세요!
25 봄 코드에서 송편이라는 말만 찾아서 다 고치면 될 듯 합니다

import { ReactComponent as MainSection6 } from "@/static/events/2024fallMainSection6.svg";
import { ReactComponent as MainStep2 } from "@/static/events/2024fallMainStep2.svg";
import { ReactComponent as MainStep3 } from "@/static/events/2024fallMainStep3.svg";
import { ReactComponent as MainTitle } from "@/static/events/2024fallMainTitle.svg";
Copy link
Contributor

Choose a reason for hiding this comment

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

[나중에 쓸 정보]
이벤트 안내 페이지의 "추석 이벤트"가 사실 svg라는 충격적인 사실..
24 가을 이벤트용 Figma 파일 초대 드렸으니 여기서 문구 바꿔서 넣으셔도 되고, 가능하다면 글자로 바꿔도 좋을 것 같습니다

Copy link
Contributor

@jinhyeonkwon jinhyeonkwon left a comment

Choose a reason for hiding this comment

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

고생하셨습니다😊
몇 가지 수정사항과 나중에 도움이 될 정보를 남겨두었습니다

value="quests"
options={[
{ value: "quests", label: "퀘스트", to: "/event/2025spring-missions" },
{
Copy link
Contributor

Choose a reason for hiding this comment

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

출석 체크 관련 파일이 복사가 되지 않은 것 같습니다! 아마 회의에서 출석 체크에 대한 논의가 이루어지지 않아서인 것 같군요..
출석 체크를 구현하려면 출석 체크 관련 파일 복사와 routing 등을 마저 설정해 주시고, 만약 출석 체크를 빼게 된다면 여기 있는 출석 체크 탭은 삭제해 주세요!

@@ -17,7 +17,7 @@ const CreditAmountStatusContainer = ({
type = "credit",
...whiteContainerProps
}: CreditAmountStatusContainerProps) => {
const { creditAmount } = useValueRecoilState("event2024FallInfo") || {};
const { creditAmount } = useValueRecoilState("event2024FallInfo") || {}; // EVENT TODO: fix if needed
Copy link
Contributor

Choose a reason for hiding this comment

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

이번 이벤트 관련된 전역 변수는 모두 새 이름으로 관리하는 게 좋을 것 같습니다!
이미 2025Spring으로 쓰신 코드도 있어서 이름 통일하면 될 것 같습니다

@@ -130,6 +131,11 @@ const ModalCredit = ({
name: t("page_credit.category_all"),
body: <BodyMembers values={members} />,
},
{
key: "2025SpringEvent",
name: t("page_credit.category_2024spring_event"),
Copy link
Contributor

Choose a reason for hiding this comment

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

이벤트 credit이 2024 이벤트 이름으로 보입니다!
i18n 적용되어 있는데, packages/web/src/pages/Mypage/langs/en.json 과 ko.json 참고하시면 금방 이해하실 것 같아요
이거 포함해서 3개 파일 업데이트 부탁드립니다!

@@ -4,7 +4,7 @@ import WhiteContainer from "@/components/WhiteContainer";

import theme from "@/tools/theme";

import { ReactComponent as CreditIcon } from "@/static/events/2023fallCredit.svg";
import { ReactComponent as CreditIcon } from "@/static/events/2023fallCredit.svg"; // EVENT TODO: fix if needed
Copy link
Contributor

Choose a reason for hiding this comment

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

[나중에 쓸 정보]
이거 송편 이미지라서, 코인 종류에 따라 바꿔야 할 것 같긴 합니다
2024 봄학기에 사용한 넙죽코인 gif 파일이, 무난하게 쓸 수 있는 좋은 에셋인 것 같아요

@malloc3141
Copy link
Contributor Author

prettier 적용 + 실수했던 것들 수정해서 푸시했습니다~ 확인 부탁드려요

Copy link
Member

@kmc7468 kmc7468 left a comment

Choose a reason for hiding this comment

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

LGTM! 수고하셨습니다

@malloc3141 malloc3141 merged commit ed16842 into dev Feb 11, 2025
4 checks passed
@malloc3141 malloc3141 deleted the 848-2025-spring-event-prepare-skeleton branch February 11, 2025 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2025 봄 이벤트: 코드 복사
3 participants