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

NHT: Users shall be able to create “stats." #345

Open
josh-bitovi opened this issue Jun 25, 2019 · 0 comments
Open

NHT: Users shall be able to create “stats." #345

josh-bitovi opened this issue Jun 25, 2019 · 0 comments
Assignees

Comments

@josh-bitovi
Copy link

josh-bitovi commented Jun 25, 2019

To expand the number of people who can create stats and reduce the amount of time that admins spend entering them users with accounts can create stats. Note that users and Players are not the same thing. A user has an account, a Player is an object in the DB. A person does not have to be a Player to be a user.

  • Users shall be logged in before they create stats (users who are not logged in cannot create stats).
  • Users shall be able to create stats for any tournament / game / player combination.
  • Stats shall be associated with the user that created them.
  • Users shall be able to delete stats they created.

Users do not have to be granted permission to add stats, they only need to create an account using the normal process. The user will not be able to do any other tasks; for example deleting games or creating Players.

Original documentation

Stats created by non-admin users ("players") should be:

  • Shown by default
  • Marked as "unapproved" or "needs approval" to admin viewers
  • Can be approved by admins, in which case it becomes a "normal" stat
  • Retain creator UID

We may also need an admin UI for listing unapproved stats... but...

  1. Spin that off into another issue, before this one is closed
  2. Allow admins to approve stats inline in the usual video player
@josh-bitovi josh-bitovi changed the title Create a Player level authorization 🛡 to create only “stats” (but not delete players, games, tournaments, etc.) NHT: Create a Player level authorization 🛡 to create only “stats” (but not delete players, games, tournaments, etc.) Jun 25, 2019
@rlmcneary2 rlmcneary2 changed the title NHT: Create a Player level authorization 🛡 to create only “stats” (but not delete players, games, tournaments, etc.) NHT: Create a User level authorization 🛡 to create only “stats” (but not delete players, games, tournaments, etc.) Jun 27, 2019
@rlmcneary2 rlmcneary2 changed the title NHT: Create a User level authorization 🛡 to create only “stats” (but not delete players, games, tournaments, etc.) NHT: Users shall be able to create “stats." Jun 27, 2019
@rlmcneary2 rlmcneary2 self-assigned this Jun 27, 2019
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

No branches or pull requests

2 participants