Skip to content
This repository has been archived by the owner on Jun 9, 2023. It is now read-only.

Update README Terminology to define and split out admin vs organizer terms #313

Merged
merged 6 commits into from
Jan 16, 2020

Conversation

allella
Copy link
Contributor

@allella allella commented Dec 30, 2019

  • I have read Chapter's contributing guidelines.
  • My pull request has a descriptive title (not a vague title like Update README.md).
  • My pull request targets the master branch of Chapter.

Closes Nothing.

See my fork for a visual of the changes.

@vkWeb
Copy link
Member

vkWeb commented Dec 31, 2019

I think we should confirm this with @QuincyLarson. If he agrees to the permissions we are giving to organizers and admin then I'll be happy to merge.

@allella
Copy link
Contributor Author

allella commented Jan 16, 2020

Using @QuincyLarson 's "prior art" of moving things along if they sit unanswered, I'm merging this in since it's more clear than what we have and it's only a README change.

We're also having a conversation on #296 which depends on setting the user_role types, which should correspond to this terminology. We'll figure out if the terminology needs tweaking in the course of defining the user_role table.

@allella allella merged commit e6f47e1 into freeCodeCamp:master Jan 16, 2020
@allella allella deleted the docs/readme-admin-vs-organizer branch January 16, 2020 09:55
@QuincyLarson
Copy link
Contributor

@allella I'm a bit late to the party, but I just want to confirm - these are nice, succinct definitions.

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