fix: duplicate email issue caused by redundant NATS subscribers

[sujitaw]

What

• Fixed multiple mails being sent during user verification

Summary by CodeRabbit

• Refactor
  • Reorganized internal module architecture for improved code structure and maintainability.

[coderabbitai]

📝 Walkthrough

Walkthrough

The EcosystemModule has been refactored to replace module-based imports with direct repository and service-based imports. OrganizationModule and UserModule are removed, and their functionality is accessed through direct provider injections of OrganizationRepository, UserRepository, UserOrgRolesService, and UserOrgRolesRepository.

Changes

Cohort / File(s)	Summary
Module Dependency Refactoring apps/ecosystem/src/ecosystem.module.ts	Replaced module-based imports (OrganizationModule, UserModule) with repository and service imports (OrganizationRepository, UserRepository, UserOrgRolesService, UserOrgRolesRepository). Updated providers array to include four new dependencies alongside existing providers.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A hop toward clarity, we say!
Modules out, repos hold the way,
Dependencies now stand so bright,
Direct injections, oh what a sight!
Cleaner and leaner, hip hooray! 🎉

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title claims to fix a 'duplicate email issue caused by redundant NATS subscribers', but the actual changes only replace module-based imports with repository/service imports without addressing NATS subscribers or email duplication logic.	Update the title to accurately reflect the actual changes, such as 'refactor: convert module imports to repository/service-based dependencies in EcosystemModule' or provide implementation details showing how these import changes fix the duplicate email issue.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/multiple_email_issue

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

• ✨ 136.113.208.247/32 (new)
• 34.170.211.100/32
• 35.222.179.152/32

Failure to add the new IP will result in interrupted reviews.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud