Skip to content

Comments

#86 Adds delete user functionality in admin panel#110

Merged
DecDuck merged 3 commits intoDrop-OSS:developfrom
Pacodastre:86-delete-users-functionality
Jun 8, 2025
Merged

#86 Adds delete user functionality in admin panel#110
DecDuck merged 3 commits intoDrop-OSS:developfrom
Pacodastre:86-delete-users-functionality

Conversation

@Pacodastre
Copy link
Contributor

No description provided.

Copy link
Member

@DecDuck DecDuck left a comment

Choose a reason for hiding this comment

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

Honestly, looks great - just a few changes here and there. Thanks so much.

@Pacodastre Pacodastre force-pushed the 86-delete-users-functionality branch from cacb169 to 658efaf Compare June 8, 2025 03:06
Copy link
Member

@DecDuck DecDuck left a comment

Choose a reason for hiding this comment

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

As discussed in Discord, we need to prevent the user from deleting themselves.

@Pacodastre Pacodastre force-pushed the 86-delete-users-functionality branch from 658efaf to 03bacbe Compare June 8, 2025 04:30
@DecDuck DecDuck merged commit 60abc03 into Drop-OSS:develop Jun 8, 2025
2 checks passed
@Pacodastre Pacodastre deleted the 86-delete-users-functionality branch June 9, 2025 00:55
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

Successfully merging this pull request may close these issues.

2 participants