Skip to content
This repository has been archived by the owner on Jul 25, 2019. It is now read-only.

Feature/forgot password #243

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from
Open

Feature/forgot password #243

wants to merge 5 commits into from

Conversation

BbqGamer
Copy link

@BbqGamer BbqGamer commented Dec 1, 2018

Short description

Added feature to reset password in the login screen

@welcome
Copy link

welcome bot commented Dec 1, 2018

Wow, thanks for that PR!

img

@prd-ev prd-ev deleted a comment Dec 1, 2018
@prd-ev prd-ev deleted a comment Dec 1, 2018
@prd-ev prd-ev deleted a comment Dec 1, 2018
@prd-ev prd-ev deleted a comment Dec 1, 2018
@@ -0,0 +1,12 @@
<link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/4.0.0/css/bootstrap.min.css" integrity="sha384-Gn5384xqQ1aoWXA+058RXPxPg6fy4IWvTNh0E263XmFcJlSAwiGgFAW/dAiS6JXm"
Copy link
Member

Choose a reason for hiding this comment

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

Jest opcja żeby nie używać bootstrapa? #201
Szczególnie w mailach...

<html lang='pl'>

<head>
<title>Pytatki - Resetowanie Hasła</title>
Copy link
Member

Choose a reason for hiding this comment

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

Dlaczego nie użyłeś extends?

<html lang='pl'>

<head>
<title>Pytatki - Resetowanie Hasła - Etap 2</title>
Copy link
Member

Choose a reason for hiding this comment

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

Tak samo jak wyżej - extends

@@ -194,6 +194,59 @@ def login_get():
return render_template('login.html')


@APP.route('/resetpassword/', methods=["GET"])
Copy link
Member

Choose a reason for hiding this comment

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

Słowa w url'u powinny być oddzielone myślnikiem

Suggested change
@APP.route('/resetpassword/', methods=["GET"])
@APP.route('/reset-password/', methods=["GET"])

return render_template('password_reset.html')


@APP.route('/resetpassword/', methods=["POST"])
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
@APP.route('/resetpassword/', methods=["POST"])
@APP.route('/reset-password/', methods=["POST"])

return render_template('password_reset_page.html')


@APP.route('/user/resetpassword/<token>', methods = ['POST'])
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
@APP.route('/user/resetpassword/<token>', methods = ['POST'])
@APP.route('/user/reset-password/<token>', methods = ['POST'])

@pniedzwiedzinski pniedzwiedzinski changed the base branch from master to develop December 2, 2018 09:15
@prd-ev prd-ev deleted a comment Dec 2, 2018
@prd-ev prd-ev deleted a comment Dec 2, 2018
@prd-ev prd-ev deleted a comment Dec 2, 2018
@prd-ev prd-ev deleted a comment Dec 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants