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

[FEAT] Authentication and Authorization #144

Merged
merged 9 commits into from
Jul 25, 2024

Conversation

Cyberguru1
Copy link
Contributor

@Cyberguru1 Cyberguru1 commented Jul 24, 2024

Description

Develop and integrate a seamless user sign up feature to enhance user engagement and onboarding experience. This feature should include user registration, account creation, and authentication processes. Incorporate user and admin sign up, user login, user logout, change password, and assign roles to users. Implement robust authentication and authorization mechanisms to ensure secure access control.

Related Issue (Link to Github issue)

Sign up issue 1

login issue 112

change password issue 135

logout issue 136

assign role issue 137

assign role issue 160

How Has This Been Tested?

All features implemented has been tested with go default test tool, and all test including previous one's written passed
the following command was used

go test ./... -timeout 99999s -v

Screenshots (if appropriate - Postman, etc):


adminlogin
adminsingup
userlogin
userlogout
usersignup

changepassword
roleassign

All test cases passed:
image
image
image
image

Participants For the authentication and authorization team:

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Copy link
Contributor

@Micah-Shallom Micah-Shallom left a comment

Choose a reason for hiding this comment

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

all is good

@Cyberguru1 Cyberguru1 requested a review from Micah-Shallom July 24, 2024 22:58
Copy link
Contributor

@Micah-Shallom Micah-Shallom left a comment

Choose a reason for hiding this comment

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

all is good

@NwokoyeChigozie NwokoyeChigozie merged commit 4eba76b into dev Jul 25, 2024
4 checks passed
@Cyberguru1 Cyberguru1 self-assigned this Aug 2, 2024
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.

4 participants