search

syndesisio / syndesis-rest

The API for Syndesis - a flexible, customizable, cloud-hosted platform that provides core integration capabilities as a service. It leverages Red Hat's existing product architecture using OpenShift Online/Dedicated and Fuse Integration Services.
https://syndesis-staging.b6ff.rh-idev.openshiftapps.com/api/v1/
Apache License 2.0
6 stars 17 forks source link

refactor(connectors): Bring back SQL stored connector #689

Closed zregvart closed 7 years ago

zregvart commented 7 years ago

This re-enables the SQL stored procedure connector and the accompanying connection by moving it from demo-data.json to deployment.json.

jimmibot commented 7 years ago

@zregvart, thanks! @KurtStam, please review this.

pure-bot[bot] commented 7 years ago

Pull request approved by @rhuss - applying approved label

KurtStam commented 7 years ago

@zregvart looks like the connectors version is set to 0.4-SNAPSHOT which makes a test fail.

zregvart commented 7 years ago

@zregvart looks like the connectors version is set to 0.4-SNAPSHOT which makes a test fail.

It might need a new connectors release, I think we have latest as the version, let me debug this a bit...

zregvart commented 7 years ago

This is an issue in SQL stored connector, I've created https://github.com/syndesisio/connectors/pull/86. This PR should be updated when we release connectors.

KurtStam commented 7 years ago

retest this please

KurtStam commented 7 years ago

@zregvart I think this may need a rebase now that the connector project is released.

zregvart commented 7 years ago

I've rebased this but I think the build will fail due to https://github.com/syndesisio/connectors/issues/88 so it will require a new version of connectors.