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

#277 업데이트 알림 뷰 #281

Merged
merged 4 commits into from
May 3, 2023
Merged

#277 업데이트 알림 뷰 #281

merged 4 commits into from
May 3, 2023

Conversation

joohaem
Copy link
Member

@joohaem joohaem commented May 2, 2023

  • 브랜치명, 브랜치 알맞게 설정
  • Reviewer, Assignees, Label, Milestone, Issue(PR 작성 후에) 붙이기
  • PR이 승인된 경우 해당 브랜치는 삭제하기

📌 내용

  • Feat/#277 update #278 PR에 리팩토링을 추가적으로 진행하였습니다!
  • 현재 dev 서버 마비로 .. 동작 확인은 못해봤습니다🤔🤔 아마 달라진 부분은 lazy loading 빼고는 없습니다.

@joohaem joohaem requested a review from NYeonK May 2, 2023 01:21
@joohaem joohaem self-assigned this May 2, 2023
@joohaem joohaem linked an issue May 2, 2023 that may be closed by this pull request
4 tasks
@joohaem joohaem changed the base branch from develop to release/1.4.0 May 2, 2023 01:21
import Medley from "./Medley";
import MoodPiickle from "./MoodPiickle";
import PiickleMe from "./PiickleMe";
import StripBanner from "./StripBanner";
import { St } from "./style";
import UpdateModal from "./UpdateModal";

const UpdateModal = lazy(() => import("./UpdateModal"));
Copy link
Member Author

Choose a reason for hiding this comment

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

전체 UI가 나온 이후에, 업데이트 모달창을 띄우고 싶었습니다.!

Copy link
Contributor

Choose a reason for hiding this comment

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

오오오오옹 새로운 거 배워갑니다:)

closeHandler: () => void;
}
export default function UpdateModal() {
const { isOpened, handleCloseModal } = useUpdateModal();
Copy link
Member Author

Choose a reason for hiding this comment

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

훅을 컴포넌트 내에 위치시켰습니다!

Copy link
Contributor

Choose a reason for hiding this comment

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

오왓 이런 방법이,,!
if (!isOpened) return null; 짱짱+-+!!!!!

Copy link
Contributor

@NYeonK NYeonK left a comment

Choose a reason for hiding this comment

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

수고하셨습니다:)

}

export default function Modal(props: PropsWithChildren<ModalContents>) {
const { theme = "DEFAULT", closeHandler, closeOpacityClassName, closeBtnClassName, children } = props;
const { theme = "DEFAULT", closeHandler, closeOpacityClassName, closeBtnClassName, isNoCloseBtn, children } = props;
Copy link
Contributor

Choose a reason for hiding this comment

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

왐마야 반영을 안했네 이고..ㅎ 거마워용>,<

closeHandler: () => void;
}
export default function UpdateModal() {
const { isOpened, handleCloseModal } = useUpdateModal();
Copy link
Contributor

Choose a reason for hiding this comment

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

오왓 이런 방법이,,!
if (!isOpened) return null; 짱짱+-+!!!!!

id: number;
content: string;
};

export const versionUpdateList: VersionUpdateList[] = [
export const VERSION_UPDATES: VersionUpdate[] = [
Copy link
Contributor

Choose a reason for hiding this comment

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

이렇게 하눈거구나~ 이거 넘나 헷갈리는고!-!

util 폴더의 비슷한 다른 파일들 참고했는데, Upper Case로 바꾸면 좋을 것들이 많을 것 같네용:)

변수명을 다 바꿨길래 궁금한건데용 파일명은 일부러 안바꾸신거겠죠?

Copy link
Member Author

Choose a reason for hiding this comment

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

@NYeonK 생각 못했어요 하하하 바꾸셔도 좋습니다 !!!!!!!!!

@joohaem joohaem merged commit 5bc46ae into release/1.4.0 May 3, 2023
@joohaem joohaem deleted the feat/#277-update branch May 3, 2023 01:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ Common ] 업데이트 알림창 구현
2 participants