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

Separate SMTP settings from the GitBucket "base" settings for the plugin #5

Closed
laaksokimmo opened this issue Sep 1, 2015 · 5 comments
Assignees
Milestone

Comments

@laaksokimmo
Copy link

SMTP settings for the announce should be splitted up:

  • SMTP settings (announce plugin)
  • SMTP notification settings ("base" GitBucket)

Doing so, we could separate the GIT notifications and it's underlying SMTP usage from any other feature that would require to send email.

We needed to remove the "base" SMTP settings and "send notifications" from the bare GitBucket configuration since it became too verbose with no ability to modity to whom and when the emails are sent out. This, naturally disables the possibility to use announce plugin.

@McFoggy
Copy link
Member

McFoggy commented Sep 1, 2015

The issue is not really announce plugin related. I created instead gitbucket/gitbucket#893.

The split has first to be done in gitbucket then we will adapt if needed.

@McFoggy
Copy link
Member

McFoggy commented Sep 2, 2015

@laaksokimmo I have pushed PR gitbucket/gitbucket#896 to answer the need.
A small change will be required in this plugin to adapt to the proposed new gitbucket settings.

@laaksokimmo
Copy link
Author

Cool, waiting takezoe to react.

@McFoggy McFoggy self-assigned this Oct 3, 2015
@McFoggy McFoggy added this to the 1.1 milestone Oct 3, 2015
@McFoggy McFoggy closed this as completed in 4b1eafd Oct 7, 2015
@laaksokimmo
Copy link
Author

I can verify this is working now, thank you for your actions.

@McFoggy
Copy link
Member

McFoggy commented Oct 12, 2015

Ok fine

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

No branches or pull requests

2 participants