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

Update allow-users-to-change-passwords.md #446

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nanowrit
Copy link
Contributor

@nanowrit nanowrit commented Feb 14, 2020

Proposing these changes to make this chapter more congruent with the core chapters of the guide.

Also, I have a replacement screenshot for the settings page with both the billing form and the change email/password buttons. Please see my comment for the screen shot.

Proposing these changes to make this chapter more congruent with the core chapters of the guide.

Also, I have a replacement screenshot for the settings page with both the billing form and the change email/password buttons. Just let me know how to get it to you.
@nanowrit
Copy link
Contributor Author

Please see attached screen shot for the settings page with both the billing form and the change email/password buttons.

Screen Shot 2020-02-14 at 11 40 36 AM

@nanowrit nanowrit requested a review from jayair February 14, 2020 18:55
@jayair
Copy link
Contributor

jayair commented Feb 23, 2020

Hmm this one is a bit of a bigger change, since we need to change the attached repos as well. I think for now it might be better to add a note in this chapter saying that we are using a blank settings page instead of the existing one?

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.

2 participants