-
Notifications
You must be signed in to change notification settings - Fork 250
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
Move idle, external, and similar features elsewhere #493
Comments
Removal part done in #494. |
For JIRA and Redmine integration, hamster-bridge already exists and looks good ! |
As a user of this, I will try to fix it :-) Starting with: notifications are not working |
@sh-zam that would be good ! |
Follow up from: PR #521 If I understand you correctly, you want the notifications to be managed separately (like in a separate service) and it should talk to hamster-service and client over dbus? |
Exactly. As stated above, this is how hamster-bridge works. |
Makes sense. If the repository is fit for hamster-bridge, I'll add it to that. |
If anybody is interested I've made some changes to hamster in my own repository https://github.com/gsobczyk/hamster:
|
Hello @gsobczyk, |
Several comments and PRs show a misunderstanding of the rationale.
So please be nice with the new maintainers. |
@u451f |
"External" was never brought back (#271),
and notifications are not working at all (tested on openSUSE 15.1).
There have been related feature requests (#123) and PRs (#118, #259) pending for a long time.
No wonder:
that kind of features is nice to have, but hard to test,
and increase a lot the maintenance burden and responsibility.
All these features can be implemented separately,
talking to the main hamster through dbus, for instance.
Let's remove them from the main repo,
and if there is an interest, eventually delegate that to a new separate project
(say
hamster-companion
), preferably with another maintainer.[I would gladly help to set it up, of course, so please ask here]
That would even be reusable with the rewrite if it resumes.
The text was updated successfully, but these errors were encountered: