Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: split Microsoft 365/Hotmail functionality #1204

Merged

Conversation

PsypherPunk
Copy link
Contributor

hotmail was previously behind the headless feature which, if disabled, breaks the Microsoft 365 API functionality.

refactor these into a microsoft module, leaving the hotmail module behind the headless feature and allowing the Microsoft 365 API to exist separately.

fixes #1203


I've verified that the Microsoft 365 API behaviour is still working in all previously-tested scenarios.

Someone more familiar with the Hotmail headless feature might want to put that through its paces though…?

`hotmail` was previously behind the `headless` feature which, if
disabled, breaks the Microsoft 365 API functionality.

refactor these into a `microsoft` module, leaving the `hotmail` module
behind the `headless` feature and allowing the Microsoft 365 API to
exist separately.
Copy link
Member

@amaury1093 amaury1093 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code LGTM, i'll do some manual tests before merging

core/src/smtp/error.rs Show resolved Hide resolved
Co-authored-by: Amaury <1293565+amaurym@users.noreply.github.com>
Copy link
Member

@amaury1093 amaury1093 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@amaury1093 amaury1093 merged commit e987b13 into reacherhq:master Oct 17, 2022
@PsypherPunk PsypherPunk deleted the 1203-hotmail-headless-feature branch October 17, 2022 17:26
juhniorsantos pushed a commit to juhniorsantos/check-if-email-exists that referenced this pull request Apr 11, 2024
* fix: split Microsoft 365/Hotmail functionality

`hotmail` was previously behind the `headless` feature which, if
disabled, breaks the Microsoft 365 API functionality.

refactor these into a `microsoft` module, leaving the `hotmail` module
behind the `headless` feature and allowing the Microsoft 365 API to
exist separately.

* fix: explanation of Microsoft365Error

Co-authored-by: Amaury <1293565+amaurym@users.noreply.github.com>

Co-authored-by: Amaury <1293565+amaurym@users.noreply.github.com>
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.

Unused Headless feature for Microsoft 365 api
2 participants