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

Rm user-specific methods from int/app/infra/port/repo #12

Closed
Tracked by #3
Sddilora opened this issue May 6, 2024 · 1 comment
Closed
Tracked by #3

Rm user-specific methods from int/app/infra/port/repo #12

Sddilora opened this issue May 6, 2024 · 1 comment
Assignees
Labels
Be Be Team compliance Ensuring adherence to standards and regulations development Issues or tasks currently in active development high-priority Urgent issues needing immediate attention sub-issue Sub-task within a larger task

Comments

@Sddilora
Copy link
Member

Sddilora commented May 6, 2024

User-specific methods:

// SaveUser insert a new user or update the existing one in the database.
SaveUser(ctx context.Context, user me.User) (me.User, error)
// DeleteUser soft-deletes the given user in the database.
DeleteUser(ctx context.Context, user me.User) (me.User, error)

// ChangePassword changes the poassword of the given user in the database.
ChangePassword(ctx context.Context, userPassword me.UserPassword) (me.UserPassword, error)

Remove user-specific methods from the repository application service.

"GetUsersByFilter" will not be removed because it is used by the "GetUserPasswordByUserId," "Login," and "Refresh" methods.

"GetUserPasswordByUserId" will not be removed because it is used by the "Login" method.

The changes will be merged into the "/first/hsm/1/create-first" branch first. Therefore, set the base branch of the pull request as /first/hsm/1/create-first.

@husamettinarabaci husamettinarabaci added the Be Be Team label May 6, 2024
@Sddilora Sddilora added compliance Ensuring adherence to standards and regulations development Issues or tasks currently in active development high-priority Urgent issues needing immediate attention Be Be Team sub-issue Sub-task within a larger task and removed Be Be Team labels May 6, 2024
@Sddilora Sddilora self-assigned this May 7, 2024
@Sddilora
Copy link
Member Author

Sddilora commented May 8, 2024

Also closed with #10

@Sddilora Sddilora closed this as completed May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Be Be Team compliance Ensuring adherence to standards and regulations development Issues or tasks currently in active development high-priority Urgent issues needing immediate attention sub-issue Sub-task within a larger task
Projects
None yet
Development

No branches or pull requests

2 participants