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

fix: superadmin update a single user #468

Open
wants to merge 1 commit into
base: clean
Choose a base branch
from
Open

fix: superadmin update a single user #468

wants to merge 1 commit into from

Conversation

max-out-oluwadara
Copy link
Contributor

@max-out-oluwadara max-out-oluwadara commented Aug 7, 2024 •
edited
Loading

Fixes Issue #149

This PR updates the AdminService to improve error handling and validation.

Changes proposed

  • Refactored error handling in the Adminservice
  • Improved validation logic to ensure data integrity

Check List

  • The endpoint should be accessible at PATCH /api/v1/admin/users/fa9adebacde...
  • The endpoint should accept HTTP PATCH requests.
  • The endpoint should require user details to be updated in the request body.
  • The endpoint should return the updated user details upon success.
  • The endpoint should return a 200 OK status code with the updated user details.

Screenshots/ Videos

Screenshot 2024-08-06 at 9 04 14 PM Screenshot 2024-08-06 at 9 05 07 PM Screenshot 2024-08-06 at 9 05 39 PM Screenshot 2024-08-06 at 9 06 26 PM Screenshot 2024-08-06 at 9 08 16 PM

@max-out-oluwadara
Merge pull request #1 from hngprojects/clean
ad62336 
fix(user): update user service for better error handling
@max-out-oluwadara max-out-oluwadara changed the title fix(user): update user service for better error handling fix: SuperAdmin update a single Aug 7, 2024
@max-out-oluwadara max-out-oluwadara changed the title fix: SuperAdmin update a single fix: superadmin update a single Aug 7, 2024
@max-out-oluwadara max-out-oluwadara changed the title fix: superadmin update a single fix: superadmin update a single user Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@max-out-oluwadara