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

bug-fx create seeders for admin and role for admin #50

Merged
merged 1 commit into from
May 1, 2024

Conversation

hozayves
Copy link
Collaborator

@hozayves hozayves commented May 1, 2024

Describe the purpose of this pull request

The aim here is to generate dummy data for both the admin user and admin role. Additionally, I intended to enhance security by implementing hashed passwords. Although I attempted to create a reusable hashedPassword function, I encountered difficulties in its utilization, yet I managed to integrate it into the auth controller and user controller.

Testing Instructions

To validate the changes:

  1. Execute the seed to generate fresh dummy data for both the admin user and role.
  2. Create an account to verify the functionality of the hash password function.

Related Issues

Missing dummy data for admin user

Checklist

Please review the following checklist and make sure all tasks are complete before submitting:

  • Code follows the project's coding standards
  • Changes are covered by tests
  • Documentation is updated (if applicable)
  • All tests pass

@hozayves hozayves added the ready label May 1, 2024
@hozayves hozayves requested a review from P-Rwirangira May 1, 2024 18:43
@hozayves hozayves force-pushed the fx-create-dummy-data-userAdmin-and-roles branch from cc8d253 to 8c837c6 Compare May 1, 2024 18:58
create reusable function for encrypt
@hozayves hozayves force-pushed the fx-create-dummy-data-userAdmin-and-roles branch from 8c837c6 to b759f77 Compare May 1, 2024 19:00
@niyontwali niyontwali merged commit 43d4d75 into develop May 1, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants