Refactoring code in src/api/groups.js

[Aya-Abdel-Hamid]

After refactoring the groupsAPI.join function to reduce cognitive complexity, the changes introduced multiple test failures, particularly with tests returning unexpected HTTP 500 status codes. The refactor aimed to streamline asynchronous operations, optimize privilege checks, and improve group event logging. However, the revised function appears to mishandle certain scenarios, leading to errors when processing group joins or membership requests. The current focus is on debugging the refactored logic to ensure it correctly handles all cases, resolves the test failures, and maintains efficient code without reintroducing high cognitive complexity.

[Aya-Abdel-Hamid]

I would like to work on this please!

[github-actions]

Great! I assigned you (@Aya-Abdel-Hamid) to the issue. Have fun working on it!

[EduardoFF]

This type of issue shouldn't be labeled as a bug. It pertains to solving a problem that affects code quality rather than functionality.