search

open-plan-tool / gui

Energy Planning Application
Other
8 stars 4 forks source link

Change/use exchange server for all mailing purposes #220

Closed 4lm closed 2 years ago

4lm commented 2 years ago

Edit: Is based on unmerged #213. Please merge #213 first.

Closes #211.

Hi @Bachibouzouk, please review this PR and merge on approval.

It now uses exchangelib for the whole app and removes all artifacts concerning Django's email integration.

Please note: Because we use exchangelib now on all occasions, we always have to overwrite Django class methods in which Django sends an email with our own implementation in app/projects/services.py:send_email. For example, Django's generic PasswordResetView (PasswordResetForm). Which I have overwritten in app/users/views.py:ExchangePasswordResetView.

Bachibouzouk commented 2 years ago

@4lm - with VPN it works :)

4lm commented 2 years ago

Hi @Bachibouzouk, from my side we can merge this PR! Would you like to merge it?