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

Migrate review block and semester block #52

Merged
merged 4 commits into from
Mar 26, 2024

Conversation

useruseruse
Copy link
Contributor

@useruseruse useruseruse commented Mar 20, 2024

  1. ReviewBlock
  • Main Page - dictionary page 로 잘 이동함
    image

  • Dictionary Page
    image

  • Planner Page
    image

  • Write Review - dictionary page 로 잘 이동함
    image

  1. ReviewSimpleBlock
  • TimeTable Page - dictionary page 로 잘 이동함
    image
  1. ReviewWriteBlock -(리뷰 작성하기 - 명예의 전당에 있음)
  • [x]
    image4. Semester Block
  • [x]
    image

@useruseruse useruseruse self-assigned this Mar 20, 2024
@useruseruse
Copy link
Contributor Author

useruseruse commented Mar 20, 2024

기타 (PR 관련된 사항은 아님)
image

Planner Page 에서 ReviewBlock 이 사용되는데 from="Dictionary" 로 되어 있고, dictionary 로 이동이 불가능함.
(from = "planner", dictionary 로 이동가능해야 하는 것 아닌지.?)

image

@useruseruse useruseruse marked this pull request as ready for review March 25, 2024 16:46
@useruseruse useruseruse added the migrate migrate from JS CC to TS FC label Mar 25, 2024
@sboh1214 sboh1214 force-pushed the migration@components/blocks1 branch from e2d4751 to 013731b Compare March 25, 2024 18:41
Copy link
Contributor

@sboh1214 sboh1214 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~!

src/components/blocks/ReviewBlock.tsx Outdated Show resolved Hide resolved
src/components/blocks/ReviewWriteBlock.tsx Outdated Show resolved Hide resolved
import Semester from '@/shapes/model/subject/Semester';
import { getSemesterName } from '@/utils/semesterUtils';

type SemesterType = Semester | 'ALL';
Copy link
Contributor

Choose a reason for hiding this comment

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

만약 이 타입이 다른 곳에서도 사용된다면 shapes 폴더에 저장하는 것이 좋을 것 같아요

src/components/blocks/SemesterBlock.tsx Outdated Show resolved Hide resolved
@sboh1214 sboh1214 changed the title fix review block and semester block Fix review block and semester block Mar 25, 2024
Copy link
Contributor

@sboh1214 sboh1214 left a comment

Choose a reason for hiding this comment

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

image

@sboh1214 sboh1214 merged commit 814f1e0 into migration Mar 26, 2024
3 checks passed
@sboh1214 sboh1214 deleted the migration@components/blocks1 branch March 26, 2024 06:17
@yumincho yumincho changed the title Fix review block and semester block Migrate review block and semester block Mar 26, 2024
@yumincho yumincho added this to the TypeScript Migration milestone Mar 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
migrate migrate from JS CC to TS FC
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants