Skip to content
This repository was archived by the owner on Nov 29, 2024. It is now read-only.

Conversation

@kgodtfredsen
Copy link

Please leave comments here with a screenshot on what you'd like changed, and I'll update as part of this PR.

@RasHanz
Copy link
Contributor

RasHanz commented Jan 26, 2022

Alright! Here are my comments:

  • I think we're missing an "a" or a "the" in all entries (several) that are like this:

  • Very nitpicky, but I think this could use a period for consistency (unless the actual error message is missing it?):

  • We're missing the plural of "action" here:

  • This should be "limited":

  • In the instances where we see this (a few) I'd suggest "The user" or "Users":

  • Here I believe "use these" is better here:

  • The word "to" wasn't deduplicated here:

  • It'd read better with a verb here:

  • A couple of articles might be better here:


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.

3 participants