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

Handle nominations with an array of nominees #7

Open
aitorres opened this issue Nov 12, 2019 · 0 comments
Open

Handle nominations with an array of nominees #7

aitorres opened this issue Nov 12, 2019 · 0 comments
Labels
enhancement New feature or request

Comments

@aitorres
Copy link
Collaborator

As a future improvement:

Currently, the nominations are handled according to their associated category's type:

  • If the category is a ONE_USER category, then mainNominee is required and must be a reference to a user id, auxNominee must NOT be present, and extra may be present
  • If the category is a TWO_USERS category, then mainNominee and auxNominee are required and must be references to user ids, and extra may be present
  • If the category is a ONLY_EXTRA category, then extra is required and its content will vary depending on the referenced category

A possible improvement, as discussed by Germán and me, would be to only have two types:

  • USERS
  • EXTRA

And change the nominees field to an array of foreign keys to user, using the fact that postgresql allows for array types.

This goes beyond the reach of Compushow 2019 but would be nice to have for other years, should the project be reused in the future.

@aitorres aitorres added the enhancement New feature or request label Nov 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant