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

Refactores the default OpenTox services, to have them as plugins #9

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

egonw
Copy link
Member

@egonw egonw commented Feb 17, 2013

This allows the adding of, for example, a local OpenTox service to be configured by a plugin. That means that users do not have to go through preferences pages, etc, but can simply install a plugin.

As a demo, a localhost plugin is provided, in addition to the earlier Ambit and Muenchen services. The latter no longer exists and the matching plugin has been removed from the feature.

…aConsult servers.

Advantages I see:

- config changes with small plugin updates
- branding at user level more visible, promoting hosting a server (possibly via feature)
- serves as template for "server" plugin
- easier en/disabling remote servers
@goglepox
Copy link
Contributor

  • The new plugins do not need an activator
  • There is no need for all the dependencies. Only net.bioclipse.opentox and even that is not strictly necessary

As is is now only the first in the preferences is used in DS see bug 3485

@egonw
Copy link
Member Author

egonw commented Feb 20, 2013

+1, thanx

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.

2 participants