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.

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

Finished implementing update organization in the organization controller #434

Merged

Conversation

ikennaumeh
Copy link
Contributor

Description

Closes #408

Changes proposed

What were you told to do?

Develop a PUT endpoint to update an existing organization.

What did you do?

  1. Added PUT /api/v1/organisations/{orgId} endpoint to Organization Controller
  2. Created a command and a command handler in the Hng.Application project to handle the interface and logic of the endpoint
  3. Added unit tests for the command handler in the Hng.Application.Test project

Check List (Check all the applicable boxes)

🚨Please review the contribution guideline for this repository.

  • My code follows the code style of this project.
  • This PR does not contain plagiarized content.
  • The title and description of the PR is clear and explains the approach.
  • I am making a pull request against the dev branch (left side).
  • My commit messages styles matches our requested structure.
  • My code additions will fail neither code linting checks nor unit test.
  • I am only making changes to files I was requested to.

Screenshots/Videos

  • Successful requests returns 200 status code
200-Success
  • Wrong user authorization/bearer token returns 403 status code
403 - Forbidden
  • No organization with the id returns 404 status code
404 - NotFound
  • All tests pass
Test passed

Copy link
Contributor

Choose a reason for hiding this comment

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

Restore this file

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@incredible-phoenix246 incredible-phoenix246 merged commit 7b4043a into hngprojects:dev Mar 1, 2025
1 check passed
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.

[feat] Update organization
2 participants