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

Changing Credited Name to Real Name on the Settings Page #3128

Merged
merged 1 commit into from
Oct 14, 2016

Conversation

BNewing
Copy link
Contributor

@BNewing BNewing commented Oct 14, 2016

At the moment, when you sign up, you're asked for your 'Real Name', but on the settings page this is referred to as 'Credited Name'. This suggested change would alter the reference on the Settings page to 'Real Name' so that they match up.

It doesn't change anything deeper than what is displayed in the web page though, and it looks like in the code in other places it might still be being referred to as credited name. At the moment, I've no idea how to change any of that.

This suggestion is a result of the glossary that Meg and I are writing - it was in defining the term in the glossary that we noticed that they're called different things in the two places.

When you sign up, you're asked to enter your Real Name, but in the Settings page that is referred to as Credited name. This change alters the Settings page reference to Real name. This only affects the UX and nothing deeper.
@BNewing BNewing added the ideas label Oct 14, 2016
@mention-bot
Copy link

@BNewing, thanks for your PR! By analyzing the history of the files in this pull request, we identified @srallen, @edpaget and @brian-c to be potential reviewers.

@simoneduca
Copy link
Contributor

Adding alumni to the mention-bot blacklist didn't seem to work then da3457c

@srallen
Copy link
Contributor

srallen commented Oct 14, 2016

@simoneduca Looks like mention bot is ignoring the config for others too: facebookarchive/mention-bot#178

@camallen
Copy link
Contributor

LGTM @BNewing

@BNewing BNewing merged commit 07ce9fa into master Oct 14, 2016
@BNewing BNewing deleted the credited-name-rename branch October 14, 2016 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants