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

[장용한] Sprint11 #317

Conversation

jangyonghan
Copy link
Collaborator

요구사항

기본

회원가입

  • 유효한 정보를 입력하고 스웨거 명세된 “/auth/signUp”으로 POST 요청해서 성공 응답을 받으면 회원가입이 완료됩니다.
  • 회원가입이 완료되면 “/login”로 이동합니다.
  • 회원가입 페이지에 접근시 로컬 스토리지에 accessToken이 있는 경우 ‘/’ 페이지로 이동합니다.

로그인

  • 회원가입을 성공한 정보를 입력하고 스웨거 명세된 “/auth/signIp”으로 POST 요청을 하면 로그인이 완료됩니다.
  • 로그인이 완료되면 로컬 스토리지에 accessToken을 저장하고 “/” 로 이동합니다.
  • 로그인/회원가입 페이지에 접근시 로컬 스토리지에 accessToken이 있는 경우 ‘/’ 페이지로 이동합니다.

메인

  • 로컬 스토리지에 accessToken이 있는 경우 상단바 ‘로그인’ 버튼이 판다 이미지로 바뀝니다.

멘토에게

@jangyonghan jangyonghan requested a review from wlgns2223 August 23, 2024 13:56
@jangyonghan jangyonghan changed the title React 장용한 sprint11 [장용한] Sprint11 Aug 23, 2024
@jangyonghan jangyonghan self-assigned this Aug 23, 2024
@jangyonghan jangyonghan added 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. 미완성🫠 죄송합니다.. labels Aug 23, 2024
@@ -0,0 +1,46 @@
import axios from "axios";

interface valuetype {
Copy link
Collaborator

Choose a reason for hiding this comment

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

valueType 보다 어떤 value인지 좀 더 명확하게 해주시면 좋을 것 같아요 !


export const postAuthSignIn = async (body: valuetype) => {
try {
const res = await axios.post(`${apiUrl}/auth/signIn`, body, {
Copy link
Collaborator

Choose a reason for hiding this comment

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

이런 부분은 axios instance로 미리 설정 할 수 있겠죠 !

Copy link
Collaborator

Choose a reason for hiding this comment

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

https://nextjs.org/docs/pages/building-your-application/routing/pages-and-layouts

nextjs layout을 이용해서 조건부 렌더링보다는 nextjs 스러운 방식으로 Header를 구성해보시는게 어떨 까요?

Comment on lines +3 to +6
const instance = axios.create({
baseURL: process.env.NEXT_PUBLIC_API_BASE_URL,
withCredentials: true,
});
Copy link
Collaborator

Choose a reason for hiding this comment

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

instance Good ! 입니다.

const [value, setValue] = useState<valuetype>(INITIAL_VALUES);
const navigate = useNavigate();

const mutation = useMutation({
Copy link
Collaborator

Choose a reason for hiding this comment

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

이것도 네이밍을 mutation보다는 어떤 일을 하는지 좀 더 명확하게 네이밍을 하시면 좋을 것 같아요 !

</button>
</form>
<div className={style.easylogin}>
간편 로그인하기
Copy link
Collaborator

Choose a reason for hiding this comment

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

p 태그로 감사주시면 좋을 것 같아요.

@wlgns2223 wlgns2223 merged commit bd484dc into codeit-bootcamp-frontend:React-장용한 Aug 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. 미완성🫠 죄송합니다..
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants