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-fe: 로그아웃 #574

Merged
merged 8 commits into from
Aug 21, 2024
Merged

feat-fe: 로그아웃 #574

merged 8 commits into from
Aug 21, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Aug 21, 2024

목적

대시보드 페이지에 로그아웃 버튼을 만듭니다.

작업 세부사항

  • 로그아웃 mutate 함수 생성
  • 로그아웃 mocking handler 생성
  • 사이드바에 로그아웃 버튼 추가

참고사항

  • Spinner에 색상 옵션을 추가했습니다.

아래의 별표줄 밑에 요구사항 ID만 작성해주세요. Prefix 금지!


LOG_OUT_01

closes #573

@github-actions github-actions bot added feature 새로운 기능 frontend 프론트엔드 labels Aug 21, 2024
@github-actions github-actions bot added this to the 스프린트 3.0 milestone Aug 21, 2024
@llqqssttyy llqqssttyy requested review from seongjinme and lurgi August 21, 2024 04:11
Copy link
Contributor Author

1724213481.171929

Copy link
Contributor Author

1724213482.173579

Copy link
Contributor Author

1724216463.018399

Copy link
Contributor

@lurgi lurgi left a comment

Choose a reason for hiding this comment

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

로그아웃 잘 확인했습니다~

@@ -10,7 +13,7 @@ const authHandlers = [
http.post(`${AUTH}/login`, async ({ request }) => {
const body = (await request.json()) as LoginFormData;

if (!body.email || !body.password || body.email !== 'admin@gmail.com' || body.password !== 'admin') {
if (!body.email || !body.password || body.email !== 'member@mail.com' || body.password !== 'qwer1234') {
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

@llqqssttyy llqqssttyy Aug 21, 2024

Choose a reason for hiding this comment

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

백엔드랑 통일시키려고 변경했어요..ㅎㅎ

Copy link
Contributor

@seongjinme seongjinme left a comment

Choose a reason for hiding this comment

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

코드 모두 체크했습니다. 감사합니다!

@@ -10,7 +13,7 @@ const authHandlers = [
http.post(`${AUTH}/login`, async ({ request }) => {
const body = (await request.json()) as LoginFormData;

if (!body.email || !body.password || body.email !== 'admin@gmail.com' || body.password !== 'admin') {
if (!body.email || !body.password || body.email !== 'member@mail.com' || body.password !== 'qwer1234') {
Copy link
Contributor

Choose a reason for hiding this comment

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

😆

@llqqssttyy llqqssttyy merged commit 2c07581 into fe/develop Aug 21, 2024
1 check passed
@llqqssttyy llqqssttyy deleted the fe-573-LOG_OUT_01 branch August 21, 2024 08:40
seongjinme pushed a commit that referenced this pull request Aug 23, 2024
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Kim Da Eun <[email protected]>
Co-authored-by: Jeongwoo Park <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 새로운 기능 frontend 프론트엔드
Projects
Status: 완료
Development

Successfully merging this pull request may close these issues.

3 participants