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

Do we still need the `setup` module? #708

Closed zregvart closed 7 years ago

zregvart commented 7 years ago

This was needed for GitHub setup but that's no longer in play, we should remove it.

gashcrumb commented 7 years ago

Does it also contain the OAuth app management stuff too? As I remember it lives at v1/setup/oauth or thereabouts...

zregvart commented 7 years ago

@gashcrumb I don't think so, that's a REST handler here: https://github.com/syndesisio/syndesis-rest/tree/master/rest/src/main/java/io/syndesis/rest/v1/handler/setup

gashcrumb commented 7 years ago

ok cool

zregvart commented 7 years ago

Would also fix #646 I think

jimmidyson commented 7 years ago

this is removed in the keycloak removal pr btw

zregvart commented 7 years ago

@jimmidyson would you mind if I remove it in another PR, would like to have it removed as soon as possible and seems straightforward?

jimmidyson commented 7 years ago

of course I don't mind, it was just a heads up