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.

Sign up for GitHub

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

Jump to bottom

Clean Roles for community moderator #511

Open
SanskarGosavi2003 opened this issue Dec 8, 2023 · 2 comments · May be fixed by #517
Open

Clean Roles for community moderator #511

SanskarGosavi2003 opened this issue Dec 8, 2023 · 2 comments · May be fixed by #517
Assignees
@arjun1131
Labels
DOPE-contribution effort0: trivial good first issue Good for newcomers model: Role priority2: important severity2: inconvenient

Comments

@SanskarGosavi2003
Copy link
Member

Overview

Currently we have two different roles for a community moderator

  1. Moderate Post
  2. Moderate Comment

They are unnecessary and the work can be done with just a single role - Community moderator

How to Solve

  1. In roles.models we have defined the roles
  2. Cooment out both the roles
  3. Add a new role for community moderator
  4. Make changes everywhere else in the code replacing those 2 roles with the new single role
@arjun1131
Copy link

Hi Sanskar,
Please assign this issue to me.

@SanskarGosavi2003
Copy link
Member Author

Hi Sanskar, Please assign this issue to me.

Sure

@sethigeet sethigeet linked a pull request Dec 17, 2023 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@arjun1131 arjun1131

Labels
DOPE-contribution effort0: trivial good first issue Good for newcomers model: Role priority2: important severity2: inconvenient
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

refactor: merge AppP and ModC body roles sethigeet/instiapp-api
2 participants
@arjun1131 @SanskarGosavi2003