Skip to content

Comments

LG-9928: Implement User suspend and reinstate methods#8578

Merged
olatifflexion merged 4 commits intomainfrom
olatif/LG-9928-user-suspension-methods
Jun 13, 2023
Merged

LG-9928: Implement User suspend and reinstate methods#8578
olatifflexion merged 4 commits intomainfrom
olatif/LG-9928-user-suspension-methods

Conversation

@olatifflexion
Copy link
Contributor

@olatifflexion olatifflexion commented Jun 12, 2023

🎫 Ticket

🛠 Summary of changes

This pull request (PR) introduces new functionality that enables the marking of a user as suspended and the ability to mark a suspended user as reinstated. Additionally, it includes the implementation of tracking analytics events.

The following methods have been added:

  • suspended?: This method allows for checking whether a user is currently suspended.
  • reinstated?: This method allows for checking whether a suspended user has been reinstated.
  • suspend: This method is used to suspend a user, marking them as suspended.
  • reinstate: This method is used to reinstate a suspended user, reversing their suspended status.

@olatifflexion olatifflexion requested a review from a team June 12, 2023 15:04
Copy link
Contributor

@n1zyy n1zyy left a comment

Choose a reason for hiding this comment

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

I think we need to handle accounts changing state more than once, but other than that this looks like a really solid foundation.

Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

LGTM

@olatifflexion olatifflexion merged commit ed6534c into main Jun 13, 2023
@olatifflexion olatifflexion deleted the olatif/LG-9928-user-suspension-methods branch June 13, 2023 20:45
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.

3 participants