Update to "Requesting changes to person data" - October 2023 #359
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
As discussed among WRT in an email thread and shared doc.
Fixes Deni's comments
|
Are you guys sure it's a good idea to link to specific people's profiles? I dunno if I'd want to have my profile having attention drawn to it. |
|
Seems completely fine to me, as this is just showing what is already public information. As Callum said, it is valuable to have an example. Additionally, the person in the example even has a wiki page so it is not like it is a person unknown to public exposure. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As discussed among WRT in an email thread and shared doc.