2 level admins - Githubissues

KamandPrompt / CodeManiacs

IIT Mandi Online Judge

MIT License

23 stars 58 forks source link

2 level admins #130

Open Vishal1541 opened 5 years ago

Vishal1541 commented 5 years ago

Currently, there is only one level of the admin who has all the privileges to add, remove other admins, problems and contests. To come up with a better approach, admins should have 2 levels:

Level1: is the supreme admin who has all the privileges to add, remove other users as admins, can see, modify, delete every problem and the contests.
Level2: is the problem setter who can just see, add, remove, modify only his problems and contests and cannot even see other admins.

dheeraj135 commented 5 years ago

@kpbot claim

kpbot commented 5 years ago

Hello @dheeraj135!

You have attempted to claim an issue without the labels "help wanted", "good first issue". It seems like you are new to KamandPrompt, so we suggest working on an issue with the help wanted or good first issue label first.

To claim this issue anyway, comment on this issue again with the command @kpbot claim --force.

dheeraj135 commented 5 years ago

@kpbot claim --force

kpbot commented 5 years ago

Welcome to KamandPrompt, @dheeraj135! We just sent you an invite to collaborate on this repository at https://github.com/KamandPrompt/CodeManiacs/invitations. Please accept this invite in order to claim this issue and begin a fun, rewarding experience contributing to KamandPrompt!

Here's some tips to get you off to a good start:

Join me on the KamandPrompt Chat, to get help, chat about this issue, and meet the other developers.
Unwatch this repository, so that you don't get 100 emails a day.

See you on the other side (that is, the pull request side)!

dheeraj135 commented 5 years ago

Hi, In the problem setter field, we are storing the name of the user. Now, the username is not guaranteed to be unique. I suggest we should store username in the problem setter field.

Benefits:

We can easily filter questions related to a particular setter. This will help us in implementing required operations for the problem setter.
Minimal code change. At max 4-5 lines needs to be changed.

Is there any particular reason to use user instead of username?

Vishal1541 commented 5 years ago

@dheeraj135 Yes, this was supposed to be done and with the previous contributors we had discussed about it but due to some reason there was no progress. You can take up this one :+1:

kpbot commented 5 years ago

Hello @dheeraj135, you have been unassigned from this issue because you have not updated this issue or any referenced pull requests for over 7 days.

You can reclaim this issue or claim any other issue by commenting @kpbot claim on that issue.

Thanks for your contributions, and hope to see you again soon!