Skip to content
This repository has been archived by the owner on Feb 2, 2022. It is now read-only.

Добавить подтверждение email при регистрации #41

Closed
jougene opened this issue Sep 21, 2019 · 10 comments
Closed

Добавить подтверждение email при регистрации #41

jougene opened this issue Sep 21, 2019 · 10 comments
Assignees
@grigori-gru
Labels
backend Improvements or additions to documentation enhancement New feature or request frontend Some work with look
Milestone
MVP

Comments

@jougene
Copy link
Contributor

jougene commented Sep 21, 2019

No description provided.

@jougene jougene added backend Improvements or additions to documentation enhancement New feature or request frontend Some work with look labels Sep 21, 2019
@grigori-gru
Copy link
Contributor

Возьму

@jougene jougene added this to the MVP milestone Sep 23, 2019
@grigori-gru
Copy link
Contributor

Хочу создать таблицу отдельную для подтверждения пароля и добавить поле verified в сущность User. Потом при регистрации пользователя будет сгенерирована ссылка с uuid на /auth/verified/:token, отпралена на почту пользователя для подтверждения, после перехода сущность подтверждения удаляется, verified меняется. Ок? Или какие-то еще варианты? Откуда у нас отправление почты будет, это как-то в конфиге нужно указать?

Всё хочу собираюсь сделать на подобие того, как здесь реализовано

@jougene
Copy link
Contributor Author

jougene commented Sep 24, 2019

Откуда у нас отправление почты будет, это как-то в конфиге нужно указать?

Будет переменная окружения, что-то типа EMAIL_FROM прописанная на хероку. В первое время предлагаю использовать https://mailtrap.io/ для тестирования.

@jougene
Copy link
Contributor Author

jougene commented Sep 24, 2019 •
edited
Loading

Хочу создать таблицу отдельную для подтверждения пароля

А отдельная таблица зачем? Давай в юзера засунем все и все.
Ну типа можно и так и так, просто я бы не создавал новую сущность.

@grigori-gru
Copy link
Contributor

Давай в одной сущности, без проблем, если уж слишком в pr избыточно будет выглядеть - поменяю

@grigori-gru
Copy link
Contributor

Я сейчас pr сделаю без самой отправки письма, она в отдельном модуле будет, я все укажу. Здесь будет общая логика переделано, тесты добавлены. Пока не понял, куда редирект должен происходить.

@grigori-gru
Copy link
Contributor

В любом случае на сервис отправки у нас мока будет в тестах

@grigori-gru
Copy link
Contributor

У нас в кофниг тогда эти параметры будут добавлены:
MAIL_HOST
MAIL_PORT
MAIL_USER
MAIL_PASSWORD
MAIL_FROM

@grigori-gru
Copy link
Contributor

Здесь что-то еще нужно доделать?

@jougene
Copy link
Contributor Author

jougene commented Oct 3, 2019

Нет, можно закрывать.

@jougene jougene closed this as completed Oct 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@grigori-gru grigori-gru

Labels
backend Improvements or additions to documentation enhancement New feature or request frontend Some work with look
Projects
None yet
Milestone
MVP
Development

No branches or pull requests
2 participants
@jougene @grigori-gru