Skip to content

Feature/popup sign-in/sign up#132

Open
marias1403 wants to merge 5 commits intomasterfrom
feature/popup-login/sign-up
Open

Feature/popup sign-in/sign up#132
marias1403 wants to merge 5 commits intomasterfrom
feature/popup-login/sign-up

Conversation

@marias1403
Copy link
Copy Markdown
Contributor

Closes #24

@vercel
Copy link
Copy Markdown

vercel bot commented Sep 18, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
uvalery ✅ Ready (Inspect) Visit Preview Sep 18, 2022 at 8:06PM (UTC)

Copy link
Copy Markdown
Contributor

@Kirill-Kor Kirill-Kor left a comment

Choose a reason for hiding this comment

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

  1. image
  2. Если залогиниться и разлогиниться - кнопка будет активной

@CyrilLaz
Copy link
Copy Markdown
Contributor

Прошу в реквесте написать что было сделано, я потыкал потыкал, у меня только открывается форма, могу ввести данные, валидация вроде работает. ввожу регистрацию. вижу alert окно что я зареган. и всё.... дальше я не смог пройти.
Что я не так делаю?

@Allxy
Copy link
Copy Markdown
Contributor

Allxy commented Sep 19, 2022

Прошу в реквесте написать что было сделано, я потыкал потыкал, у меня только открывается форма, могу ввести данные, валидация вроде работает. ввожу регистрацию. вижу alert окно что я зареган. и всё.... дальше я не смог пройти. Что я не так делаю?

Нужна почта валидная для входа и пароль - не менее 8 символов, только латиница, минимум 1 спец символ, минимум 1 заглавная, минимум 1 строчная

@CyrilLaz
Copy link
Copy Markdown
Contributor

как всё сложно, зачем усложнять так жизнь юзерам без предупреждения?) да это я затупил, не догадался)

@marias1403
Copy link
Copy Markdown
Contributor Author

как всё сложно, зачем усложнять так жизнь юзерам без предупреждения?) да это я затупил, не догадался)

Если хочешь, помоги сделать валидацию, которая расширяет стандартные validate message, тогда будет понятно 😂

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