-
Notifications
You must be signed in to change notification settings - Fork 12
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
Combine Jira adapters #359
Comments
Hmm, we could also simply exclude the server adapter from any |
Also an option @andreasknoepfle 👍 I just put down the current state of the discussion in the issue. Didn't want to exclude any other options. |
As it turns out, Atlassian is discontinuing Jira Server anyways: https://www.atlassian.com/migration/assess/journey-to-cloud We could mark the Jira Server adapter as "legacy" and at some point completely drop it. |
Does not say anything about how long people may still be running the software though... |
While working on #349 we split out the Jira adapter into two separate adapters,
jira-server
andjira-cloud
in #354. However, due to tickety-tick going through all adapter for a page concurrently and the deciding factor for thejira-server
adapter also works for Jira cloud, we can't guarantee that thejira-server
adapter will only apply for Jira Server instances, but also for Cloud.This is fine for now, as the extension is not "broken" for Jira any more. Still, it would make sense to fix that with a more deterministic solution. See this comment for some background.
The idea is: combine the two Jira adapters in one and bake the decision for which API to use in there. (Note: this does not mean there can't be two Jira adapters -
search
should just talk to one singular entry point for Jira).The text was updated successfully, but these errors were encountered: