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

로그인/회원가입 구현 #4

Open
wants to merge 9 commits into
base: develop
Choose a base branch
from
Open

로그인/회원가입 구현 #4

wants to merge 9 commits into from

Conversation

yoonali
Copy link
Contributor

@yoonali yoonali commented Apr 19, 2023

[생성]
src > page들

[오류]
회원가입이 가능해서 오류가 안나고 db에 잘 생성된다.
로그인으로 와서 가입한 회원으로 로그인을 하면 204에러가 뜨게 된다.

[사용 로직]
javascript. phpmyadmin(xampp) - mysql

테이블 : Signup > login

실행은 다 되지만 로그인을 하면 가입한 계정도 204에러가 뜨면서 alert창이 띄워짐...
이걸 수정하면 또 가입하지 않은 계정도 모두 통과가 되면서 /home 페이지로 이동됨..
Copy link
Member

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.

Copy link
Member

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.

삭제했습니다!

src/index.js Outdated

const root = ReactDOM.createRoot(document.getElementById('root'));
root.render(
<React.StrictMode>
Copy link
Member

Choose a reason for hiding this comment

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

해당 태그 사용 안 함.

Copy link
Contributor Author

@yoonali yoonali Apr 22, 2023

Choose a reason for hiding this comment

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

삭제했습니다!

src/index.js Outdated
Comment on lines 15 to 18
// If you want to start measuring performance in your app, pass a function
// to log results (for example: reportWebVitals(console.log))
// or send to an analytics endpoint. Learn more: https://bit.ly/CRA-vitals
reportWebVitals();
Copy link
Member

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.

삭제했습니당!

e.preventDefault();
seterrors(Vaildation(values));
if (errors.email === "" && errors.password === "") {
axios.post('http://localhost:8081/login', values)
Copy link
Member

Choose a reason for hiding this comment

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

Server Host 이상함.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

<div className='d-flex justify-content-center align-items-center bg-primary vh-100'>
<div className='bg-white p-3 rounded w-25'>
<h2>Login</h2>
<form action='' onSubmit={handleSubmit}>
Copy link
Member

Choose a reason for hiding this comment

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

form Tag 사용 X

Copy link
Contributor Author

Choose a reason for hiding this comment

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

form 태그 사용 대신 button 태그에 onclick 기능을 넣어주었습니다!

<div className='d-flex justify-content-center align-items-center bg-primary vh-100'>
<div className='bg-white p-3 rounded w-25'>
<h2>Sign-up</h2>
<form action='' onSubmit={handleSubmit}>
Copy link
Member

Choose a reason for hiding this comment

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

form Tag 사용 X

pinomaker-hoo and others added 7 commits April 20, 2023 10:30
드디어 로그인과 회원가입 모두 가능합니다!!!!왕 !!!!
필요없는 파일 삭제
post 했을때 200이 떴지만 400에러가 뜨면서 사용자를 찾을 수 없다고 에러가 발생
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.

2 participants