Skip to content

Conversation

douglasduteil
Copy link
Contributor

@douglasduteil douglasduteil commented Sep 11, 2025

UpdateUserByIdInOrganization

@douglasduteil douglasduteil self-assigned this Sep 11, 2025
@github-actions github-actions bot added 🌐 api Global, connected routes handling requests. scope: moderations scope: users 🗄️ database labels Sep 11, 2025
@douglasduteil douglasduteil force-pushed the douglasduteil/refactor-moderation-rename-UpdateUserByIdInOrganization branch from a3892bf to 5c97df2 Compare September 11, 2025 12:14
@douglasduteil douglasduteil changed the title ♻️ replace direct DB query with GetModerationWithUser repository ♻️ refactor UpdateUserByIdInOrganization to factory pattern Sep 11, 2025
@douglasduteil douglasduteil force-pushed the douglasduteil/refactor-moderation-rename-UpdateUserByIdInOrganization branch from 5c97df2 to 10121d9 Compare September 11, 2025 12:24
- Rename update_user_by_id_in_organization.ts to UpdateUserByIdInOrganization.ts
- Convert function to factory pattern following CONTRIBUTING.md conventions:
  - Factory function: UpdateUserByIdInOrganization (PascalCase)
  - Returned function: update_user_by_id_in_organization (snake_case)
  - Add handler type export for dependency injection
- Update controller to use factory pattern with proper dependency injection
- Update repository index exports

This aligns with the established UseCase convention and improves consistency.
@douglasduteil douglasduteil force-pushed the douglasduteil/refactor-moderation-rename-UpdateUserByIdInOrganization branch from 10121d9 to 22a7d49 Compare September 11, 2025 12:24
@douglasduteil douglasduteil merged commit c97ccdc into master Sep 11, 2025
30 of 45 checks passed
@douglasduteil douglasduteil deleted the douglasduteil/refactor-moderation-rename-UpdateUserByIdInOrganization branch September 11, 2025 12:33
Copy link

🚀 This pull request is included in v2025.9.0. See Release 2025.9.0 for release notes.

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.

1 participant