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

issue #34 Delete Gmail service. #144

Merged
merged 3 commits into from
Feb 14, 2018
Merged

Conversation

Myroslavv
Copy link
Contributor

@Myroslavv Myroslavv commented Feb 14, 2018

Summary

Delete GmailAccountMailService.cs.

closes #34

Areas Of Interest

None

Checklist

Design

None

Perfromance

None

Unit Tests

None

@Myroslavv Myroslavv requested a review from a team as a code owner February 14, 2018 13:58
container.Register<IMailService, GmailAccountMailService>(IocLifetimeEnum.Scoped);
}
else
else if (Is<IISDeployment>.Disabled)
Copy link
Member

Choose a reason for hiding this comment

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

Now we might have potential problem. If either IISDeployment and DebugMode flags are not set properly we won't have IMailService registered and app will fail.

Make sure DebugMailService is registered if nothing else is registered.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

default

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do you mean this?

Copy link
Member

Choose a reason for hiding this comment

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

yes

@Myroslavv Myroslavv changed the title Remove GmailAccountMailService.cs. Edite IOCUIModule corresponding to… issue #34 Delete Gmail service. Feb 14, 2018
@sdiachen sdiachen merged commit 1ae37af into master Feb 14, 2018
@sdiachen sdiachen deleted the issue_#34_remove_gmail_service branch February 14, 2018 15:21
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.

Remove GMail based mail service
2 participants