Skip to content
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

User registration styling #17

Merged
merged 30 commits into from
Mar 16, 2018
Merged

User registration styling #17

merged 30 commits into from
Mar 16, 2018

Conversation

richpeters
Copy link

No description provided.

@ikonitas ikonitas changed the base branch from user-registration to master March 13, 2018 16:04
@@ -0,0 +1,103 @@
import React from 'react';
Copy link
Member

Choose a reason for hiding this comment

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

Can we change this path into something else to be more recognizable for what it is? Example src/apps/accounts/registration.jsx ?

@ikonitas
Copy link
Member

  1. Header Apply for access instead of Apply for account
  2. Can we match top nav as per invision? https://projects.invisionapp.com/share/58G0EWTWZ6J#/screens/281122362

@ikonitas ikonitas assigned richpeters and unassigned ikonitas Mar 14, 2018
@ikonitas
Copy link
Member

@richpeters not sure whats going on here but pages aren't loading?

@ikonitas
Copy link
Member

ikonitas commented Mar 15, 2018

  1. http://127.0.0.1:8000/accounts/registration/ still not matching a top nav as per https://projects.invisionapp.com/share/58G0EWTWZ6J#/screens/281122362
  2. http://127.0.0.1:8000/accounts/login/ Please replace Problems logging in? with the Forgo password? it sounds like it will open report page to submit the issue instead of forgot password form.
  3. http://127.0.0.1:8000/accounts/login/ Change Submit button name to Login
  4. http://127.0.0.1:8000/accounts/password-reset/ stopped working

@richpeters
Copy link
Author

richpeters commented Mar 15, 2018

Webpack Node SASS known issue. end runserver process, start again, save .scss file, refresh page

@ikonitas
Copy link
Member

ikonitas commented Mar 15, 2018

  1. http://127.0.0.1:8000/accounts/registration/ change Register to Login at the top.
  2. http://127.0.0.1:8000/accounts/password-reset/done/ needs designing.
  3. Try to reset password for your email address you will recevie email in the console url should look like this http://127.0.0.1:8000/accounts/reset/MzU/set-password/ needs designing.
  4. http://127.0.0.1:8000/accounts/reset/done/ needs desinging.

@ikonitas
Copy link
Member

Can I get the URL to the known issue for the webpack?

@richpeters
Copy link
Author

richpeters commented Mar 15, 2018

webpack-contrib/sass-loader#100 has something to do with importing files. Suggested that you use env UV_THREADPOOL_SIZE=100 but not sure @ZATAN

richpeters and others added 11 commits March 16, 2018 09:34
# Conflicts:
#	apps/accounts/forms.py
#	apps/accounts/urls.py
#	static/src/scss/_resources.scss
#	static/src/scss/global/_navigation.scss
#	static/src/scss/styles.scss
#	templates/base.html
#	templates/group.html
#	templates/includes/header.html
#	templates/profile.html
#	templates/test.html
#	webpack.config.js
#	webpack.production.js
Copy link
Member

@tomkins tomkins left a comment

Choose a reason for hiding this comment

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

I hope @ZATAN won't kill me!

@richpeters richpeters merged commit 12e01ae into master Mar 16, 2018
@richpeters richpeters deleted the user-registration-styling branch March 16, 2018 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants