-
Notifications
You must be signed in to change notification settings - Fork 14
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
Data center compatibility #9
Comments
I suspect the plugin will work on Datacenter distributions just fine. The problem with making it officially compatible is that I don't have a way of testing it and making sure that it always stays compatible. |
Makes sense. I'll give it a shot and see how it goes. The Confluence documentation makes it sound easy to test plugins on a cluster. I suspect it's similar for Stash but I don't know enough about it. |
Has anyone tried to modify this plugin for Data Center compatibility? |
To my knowledge no, but it may work in that environment. I don't have one to test with, sadly. |
The plugin did install on Data Center but with a warning and "INCOMPATIBLE" badge. I'm planning on testing and will report back here if I find any issues. |
I just created a PR (#21) for this which fixes a potential issue fetching plugin configuration given a Data Center setup. |
@monitorjbl Can we get PR #21 merged to support the Data Center users? |
Any idea what it'd take to make this plugin cluster or data center compatible? Looking at the code, settings are persisted using
PluginSettings
. That seems to save to the database which would be safe and usable for a cluster.The text was updated successfully, but these errors were encountered: