Skip to content

Commit 9b142ac

Browse files
fixed login bug
1 parent 025b16c commit 9b142ac

File tree

3 files changed

+22
-8
lines changed

3 files changed

+22
-8
lines changed

src/routes/LoginPage/LoginPage.css

+5
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,11 @@
1010
font-family: 'Lato', sans-serif;
1111
}
1212

13+
.login-page__login-error {
14+
color: red;
15+
padding-bottom: 10px;
16+
}
17+
1318
.login-page__username,
1419
.login-page__password {
1520
text-align: right;

src/routes/LoginPage/LoginPage.js

+8-3
Original file line numberDiff line numberDiff line change
@@ -32,12 +32,13 @@ class LoginPage extends Component {
3232
user_name.value = ''
3333
password.value = ''
3434
TokenService.saveAuthToken(res.authToken)
35+
this.handleLoginSuccess()
3536
})
3637
.catch(res => {
37-
this.setState({ error: res.error })
38+
user_name.value = ''
39+
password.value = ''
40+
this.setState({ error: true })
3841
})
39-
40-
this.handleLoginSuccess()
4142
}
4243

4344
handleLoginSuccess = () => {
@@ -50,10 +51,14 @@ class LoginPage extends Component {
5051
}
5152

5253
render () {
54+
const loginFailed = this.state.error
55+
? <div className='login-page__login-error'>Incorrect username or password</div>
56+
: null
5357

5458
return (
5559
<div className='login-page'>
5660
<h2>Login</h2>
61+
{loginFailed}
5762
<form
5863
className='login-page__form'
5964
onSubmit={this.handleSubmitJwtAuth}>

src/services/auth-api-service.js

+9-5
Original file line numberDiff line numberDiff line change
@@ -9,11 +9,15 @@ const AuthApiService = {
99
},
1010
body: JSON.stringify(credentials),
1111
})
12-
.then(res =>
13-
(!res.ok)
14-
? res.json().then(e => Promise.reject(e))
15-
: res.json()
16-
)
12+
.then(res => {
13+
if (!res.ok) {
14+
return res.json()
15+
.then(error => {
16+
throw error
17+
})
18+
}
19+
return res.json()
20+
})
1721
},
1822
}
1923

0 commit comments

Comments
 (0)