Skip to content

[7주차] Pounder 미션 제출합니다.#8

Open
usernamebuzz wants to merge 75 commits intoCEOS-Developers:mainfrom
founder-dev:main
Open

[7주차] Pounder 미션 제출합니다.#8
usernamebuzz wants to merge 75 commits intoCEOS-Developers:mainfrom
founder-dev:main

Conversation

@usernamebuzz
Copy link

@usernamebuzz usernamebuzz commented Jun 24, 2022

안녕하세요 파운더 프론트 오민지, 유세은입니다. 기능 구현 때문에 리팩토링 및 타입스크립트 적용은 하지 못해 아쉽습니다😢 디자인은 기능 구현하면서 막힐 때 마다 틈틈이 했습니다. Glassmorphism을 사용해보고 싶었는데 디자인은 어렵네요🤣

배포 링크
https://pounder-vote-six.vercel.app/

구현한 기능
회원가입: 기존에 가입된 아이디나 이메일을 입력하면 가입할 수 없습니다.
로그인: 회원가입 한 아이디와 비밀번호 혹은 백엔드에 저장된 아이디와 비밀번호로 로그인이 가능합니다
투표: 한 아이디 당 두 명만 투표 가능합니다. 로그인 하지 않은 상태에서 투표 시도 시 로그인을 해야한다는 alert가 뜹니다.

구현 중인 기능
로그아웃: 로그아웃 post를 보내는 것 까지는 성공했는데 로그인이 유지되는 버그가 있습니다.
로그인을 하지 않은 상태에서 로그아웃을 시도하면 로그인을 먼저 해야한다는 alert가 뜹니다.

백엔드와의 협업
Postman을 통해 소통했습니다. 프론트에서 토큰 오류 발생 시 postman을 통해 프론트의 오류인지 판단할 수 있었습니다. Access token, refresh token, authorization 등 기본 개념을 배울 수 있었습니다.

프론트끼리 협업
저번 과제 수행과 이번 과제 초반 수행 시에는 따로 기능을 구현해서 기능별 브랜치를 만들지 않아도 괜찮았는데, 이번 과제에서는 같은 기능을 동시에 구현하거나 고칠 일이 많아서 기능별 브랜치의 중요성을 깨달을 수 있었습니다..^^

고친 버그
백에서 설정한 변수와 맞지 않아 생긴 버그
흰 인풋에 흰 글자를 설정해서 입력이 안된다고 착각한 버그

참고 자료:
https://meetup.toast.com/posts/92
https://axios-http.com/docs/api_intro
https://cloud.google.com/api-gateway/docs/authenticating-users-jwt#:~:text=To%20authenticate%20a%20user%2C%20a,API%20to%20process%20the%20authentication

YoommY2 and others added 30 commits June 21, 2022 02:43
Fix: Axios works with another data
Style: Apply linear-gradient
Style: Add style to candidate list
@corinthionia
Copy link
Member

corinthionia commented Jun 26, 2022

두 분 다 백엔드와 협업하는 과정이 이번이 처음이라 알고 있는데 과제를 잘 수행해 주셨네요! 다만 투표 기능이 제대로 동작하지 않는 점이 조금 아쉽습니다 🥲 가능하다면 과제 시연 때 이 부분을 해결해서 보여 주시면 더욱 좋을 것 같아요~ 또한 api의 base URL 같은 경우는 환경변수로 관리하시는 게 보안적인 측면에서 더 좋을 것 같습니다.
과제 하시느라 정말 고생 많으셨습니다. 발표 기대하겠습니다 😀

@usernamebuzz
Copy link
Author

usernamebuzz commented Jun 26, 2022

두 분 다 백엔드와 협업하는 과정이 이번이 처음이라 알고 있는데 과제를 잘 수행해 주셨네요! 다만 투표 기능이 제대로 동작하지 않는 점이 조금 아쉽습니다 🥲 가능하다면 과제 시연 때 이 부분을 해결해서 보여 주시면 더욱 좋을 것 같아요~ 또한 api의 base URL 같은 경우는 환경변수로 관리하시는 게 보안적인 측면에서 더 좋을 것 같습니다. 과제 하시느라 정말 고생 많으셨습니다. 발표 기대하겠습니다 😀

투표 API가 중간에 바뀌어서 그런 것 같습니다. 지금 수정해서 정상 작동됩니다!

@corinthionia
Copy link
Member

로그아웃 문제는 로그아웃 버튼을 눌렀을 때 로컬스토리지를 비워 주면 해결될 것 같아요~
로그아웃을 해도 로컬스토리지에 토큰값이 남아 있어서 투표가 가능한 것 같네요

@usernamebuzz
Copy link
Author

usernamebuzz commented Jun 26, 2022

로그아웃 문제는 로그아웃 버튼을 눌렀을 때 로컬스토리지를 비워 주면 해결될 것 같아요~ 로그아웃을 해도 로컬스토리지에 토큰값이 남아 있어서 투표가 가능한 것 같네요

.then((response) => {
        console.log(response.data);
        localStorage.clear();
        window.alert('로그아웃완료');
        console.log(user);

이렇게도 해봤었는데 콘솔에 로그인했던 유저가 나타납니다..!

.then((response) => {
console.log(response.data);
window.alert('로그아웃완');
localStorage.setItem('username', ' ');
Copy link
Member

Choose a reason for hiding this comment

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

제가 이 부분을 변경했더니 로그아웃이 정상 작동하는데 한 번 해 보시겠어요?

Suggested change
localStorage.setItem('username', ' ');
localStorage.clear();

@usernamebuzz usernamebuzz force-pushed the main branch 2 times, most recently from ffec982 to 06538f4 Compare June 28, 2022 01:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants