Skip to content

edit User model and handlers for isActive status#1452

Merged
trillium merged 1 commit intohackforla:developmentfrom
bkmorgan3:user_isActive
Jul 28, 2023
Merged

edit User model and handlers for isActive status#1452
trillium merged 1 commit intohackforla:developmentfrom
bkmorgan3:user_isActive

Conversation

@bkmorgan3
Copy link
Member

@bkmorgan3 bkmorgan3 commented Jul 26, 2023

Fixes #1374

What changes did you make and why did you make them ?

  • Updated the User Model with an isActive boolean, defaulted to true
  • Implemented a Switch on the User Details section to toggle users active status
  • created handlers to update the DB of User's active state
Visuals after changes are applied

Screenshot 2023-07-25 at 2 10 17 PM

@github-actions
Copy link

Want to review this pull request? Take a look at this documentation for a step by step guide!

From your project repository, check out a new branch and test the changes.

git checkout -b bkmorgan3-user_isActive development
git pull https://github.com/bkmorgan3/VRMS.git user_isActive

Copy link
Contributor

@MattPereira MattPereira left a comment

Choose a reason for hiding this comment

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

@bkmorgan3 appreciate your being an isActive:true developer on this project 🎉

trillium

This comment was marked as duplicate.

Copy link
Member

@trillium trillium left a comment

Choose a reason for hiding this comment

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

Great work, I looked at the previous pr (1451) and agree with these changes as well as @MattPereira 's comment.

@trillium trillium merged commit 02da971 into hackforla:development Jul 28, 2023
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.

Add an "isActive" field to the user model

3 participants