Skip to content

Conversation

@bizmarcin
Copy link
Collaborator

Działają funkcjonalności generowania linku do zmiany hasła oraz sam formularz zmiany zmiany hasła. Nie ma jeszcze wysyłania na maila. Muszę to jeszcze przemyśleć. Możecie się pobawić funkcjonalnością przechodząc ze strony logowania do Odzyskaj Hasło. Następnie w pliku email_test.txt zapisywany jest link. Link powinien być aktywny przez godzinę od wygenerowania, kolejny link można wygenerować po upływie godziny.

Wydaje mi się że warto jeszcze dodać deaktywacje linku bezpośrednio po poprawnej zmianie hasła.

return row['email']
return False

def get_email_address(email):
Copy link
Collaborator

Choose a reason for hiding this comment

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

Zduplikowana funkcja

<form action="/pass_rec" method="POST">

<h2>Odzyskiwanie hasła użytkownika</h2>

Copy link
Collaborator

Choose a reason for hiding this comment

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

Brakuje flashowania z errorem, kiedy podany email jest błędny.

@visse0001
Copy link
Collaborator

Działają funkcjonalności generowania linku do zmiany hasła oraz sam formularz zmiany zmiany hasła. Nie ma jeszcze wysyłania na maila. Muszę to jeszcze przemyśleć. Możecie się pobawić funkcjonalnością przechodząc ze strony logowania do Odzyskaj Hasło. Następnie w pliku email_test.txt zapisywany jest link. Link powinien być aktywny przez godzinę od wygenerowania, kolejny link można wygenerować po upływie godziny.

Działa w porządku :) Mam tylko małą uwagę dot. flashowania w komentarzu powyżej.
Zastanawiam się, czy zmergujemy te zmiany - wiążą się z kosztami po 30-dniowym trialu.

Wydaje mi się że warto jeszcze dodać deaktywacje linku bezpośrednio po poprawnej zmianie hasła.

Też myślę, że warto.

f.write(f"http://127.0.0.1:5000/pass_rec/{sha}\n")
return render_template('base/login.html', error=message)

error = 'Invalid login or email'
Copy link
Collaborator

Choose a reason for hiding this comment

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

Proponuję zmienić komunikat na polski

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.

3 participants