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

[FE] 아티클 에디터 페이지를 제작한다. #993

Merged
merged 12 commits into from
Nov 30, 2024

Conversation

healim01
Copy link
Contributor

@healim01 healim01 commented Nov 25, 2024

❗ Issue

✨ 구현한 기능

아티클 에디터 페이지가 추가되었습니다~!

스크린샷 2024-11-25 오후 1 29 03

아티클을 추가하려고 할때마다 포스트맨으로 하는 것이 번거롭다는 이야기를 반영하여 방끗의 백 오피스가 생겼습니다.
아직 어드민 로직이 백엔드에 없다고 들어서 완전히 어드민 구분을 하여 들어갈 수 있는 로직은 아니지만 방끗 팀을 위한 이스터에그로 어드민 페이지는 구성해놨으니... 구경하세요 우하하

아 그리고 썸네일 파일 업로드는,, 생각보다 품이 많이 들기도 하고 사실 수정 api 도 현재 없는 상태라서 백엔드와 회의를 한차례하고 추가적으로 진행되어야 할거 같습니다. 현재는 일단 빈값으로 보내주고 있습니다.

여러분의 아티클 작성이 쉽기를 바랍니다...~~~!!

📢 논의하고 싶은 내용

🎸 기타

Copy link

Copy link

Copy link

@healim01 healim01 requested review from skiende74 and ooherin and removed request for skiende74 November 25, 2024 08:15
Copy link
Contributor

@skiende74 skiende74 left a comment

Choose a reason for hiding this comment

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

에디터 페이지 수고 많으셨습니다 !!

Comment on lines +24 to +25
admin: '/admin',
articleEditor: '/isHaileyGod',
Copy link
Contributor

@skiende74 skiende74 Nov 27, 2024

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.

ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ

Comment on lines +17 to +19
export const postArticle = async (article: ArticlePostForm) => {
const response = await fetcher.post({ url: BASE_URL + ENDPOINT.ARTICLES, body: article });
return response;
Copy link
Contributor

@skiende74 skiende74 Nov 27, 2024

Choose a reason for hiding this comment

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

return할때 as 타입단언 해주면 좋을거같아요

Copy link
Contributor Author

Choose a reason for hiding this comment

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

앗 네넹 ~!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

앗 그런데 이부분이 post response 라....! 타입이 따로 없네요
res 를 리턴하지 말까요?

<Outlet />
</Suspense>
<MobileLayout>
<Suspense>
Copy link
Contributor

Choose a reason for hiding this comment

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

이건 반영이 안되어도 상관 없는데
Suspense에 fallback='Loading...' 정도로만 넣어줘도 괜찮을 거 같아요

Copy link
Contributor Author

@healim01 healim01 Nov 30, 2024

Choose a reason for hiding this comment

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

근데 그게 들어가면.... 이제 이동할 때마다 로딩 워딩이 보이고 있어서 아예 이후에 스켈레톤 작업을 하거나 해야할거 같아요

넣어봤더니 생각보다 더 거슬리네욜... 한번 이야기해볼까요??

Comment on lines +24 to +25
admin: '/admin',
articleEditor: '/isHaileyGod',
Copy link
Contributor

Choose a reason for hiding this comment

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

ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ

return (
<S.PageWrapper>
<S.QuestionBox>
<S.QuestionText>Is Hailey God?</S.QuestionText>
Copy link
Contributor

Choose a reason for hiding this comment

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

여기도 갓있다ㅋㅋㅋㅋㅋ인죵

@ooherin
Copy link
Contributor

ooherin commented Nov 29, 2024

라우터 충돌 해결이랑 edit 페이지 route 변수값만 바꿔주시면 될것 같습니다!! 고생하셨습니다:)

@healim01
Copy link
Contributor Author

지금 백엔드에서 어드민 인증 처리 작업 중이라서 어드민 인증 완료되면 재작업,,, 하거나 이 피알에서 수정해서 올릴...까 했는데 너무 오래열려있어서 머지하고 이후 작업하겠습니다~!

Copy link

@healim01 healim01 merged commit 523acf0 into dev-fe Nov 30, 2024
3 checks passed
@healim01 healim01 deleted the feat/989-markdown-editor-page branch November 30, 2024 04:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants