-
Notifications
You must be signed in to change notification settings - Fork 122
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
Close #449 #481
Close #449 #481
Conversation
Уже есть ПРчик #467 Предлагаю подождать @Heaven-Tonight с фиксом линтера. |
О как |
@fey, кажется это решение работает корректно и оно, на мой взгляд, по-лучше моего, можно его тогда замерджить. А я что-то прошляпила момент с воркфлоу, в следующий раз внимательнее буду) Еще и БД на Рендере пала смертью храбрых) |
ок, тогда поревьювим этот ПР |
@@ -49,6 +49,7 @@ function SignInForm({ onSuccess = () => null }) { | |||
email: preparedValues.email, | |||
password: values.password, | |||
}); | |||
localStorage.removeItem('guestUserData'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Предлагаю перенести это в auth.signIn()
2490845
to
6022a6f
Compare
внёс изменения |
move claer localstorate in signin provider
6022a6f
to
c96ca70
Compare
кажется, я все исправил |
Задеплойте демку плз |
у вас в ПРе какой-то левый файл. |
@bobronaud тыц) Тут есть лишнее в ПРе |
Добавил возможность войти в существующий аккаунт из гостевого аккаунта
Демо - https://bobronaud-runit2.onrender.com
Ишьюс - #449