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

45 merritt admin delete #49

Merged
merged 8 commits into from
Apr 12, 2023
Merged

45 merritt admin delete #49

merged 8 commits into from
Apr 12, 2023

Conversation

wastilla
Copy link
Contributor

@wastilla wastilla commented Apr 12, 2023 •
edited
Loading

Implement delete functionality for admin users

  • Create backend service and API route for delete
  • Implement frontend service and button, available only for admin users

wquadland
wquadland requested changes Apr 12, 2023
View reviewed changes
backend/services/post.py Outdated Show resolved Hide resolved
aayush110
aayush110 requested changes Apr 12, 2023
View reviewed changes
Copy link
Contributor

@aayush110 aayush110 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey Will, this looks good. One UI-thing that I would recommend changing is changing the "back" button to something more intuitive like "view all forum posts".

wquadland
wquadland approved these changes Apr 12, 2023
View reviewed changes
Copy link
Contributor

@wquadland wquadland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

aayush110
aayush110 approved these changes Apr 12, 2023
View reviewed changes
Copy link
Contributor

@aayush110 aayush110 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@wastilla wastilla merged commit dfacb27 into stage Apr 12, 2023
@wastilla wastilla linked an issue Apr 13, 2023 that may be closed by this pull request
As Merrit Manager I want to be able to delete posts that do not follow community guidelines or are inappropriate in any way #45
Open
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aayush110 aayush110 aayush110 approved these changes

@wquadland wquadland wquadland approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@wastilla @wquadland @aayush110 @angelinasu57