Skip to content

feat: add email module with API, services and UI#28

Closed
Piya-Boy wants to merge 1 commit intojhd3197:mainfrom
Piya-Boy:feature/email
Closed

feat: add email module with API, services and UI#28
Piya-Boy wants to merge 1 commit intojhd3197:mainfrom
Piya-Boy:feature/email

Conversation

@Piya-Boy
Copy link

@Piya-Boy Piya-Boy commented Mar 3, 2026

Made-with: Cursor

@jhd3197
Copy link
Owner

jhd3197 commented Mar 4, 2026

Hey @Piya-Boy, funny timing here. I didn't notice we were both working on the email module at the same time until your PR showed up. I already had a partial implementation going but honestly your approach was way better, splitting everything into dedicated services (Postfix, Dovecot, DKIM, SpamAssassin, Roundcube) instead of one big monolithic service was the right call.

So I ended up taking your architecture as the base and merging it with what I already had, keeping your service separation, models and API structure. I also extended it with the rest of the stuff I was working on (SSO, API keys, invitations, etc) and it all landed together in #29.

I updated the roadmap to mark email as completed and you're credited in the PR. I won't be merging this one directly since it's already integrated but wanted to make sure you know your work made the final result way better. Solid work, thanks for jumping in.

@jhd3197 jhd3197 closed this Mar 4, 2026
@Piya-Boy
Copy link
Author

Piya-Boy commented Mar 4, 2026 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants