Import/Export: Existing Integration Notification

This is a…

[ ] Feature request
[ x ] Regression (a behavior that used to work and stopped working in a new release)
[ x ] Bug report  
[ ] Documentation issue or request

The problem

It used to be that importing an integration, whether it already existed or not would result in basic details of that integration appearing in the “Import Results” section. It appears that a change has been made, such that the integration details only appear in the API response of an import if it’s a newly created/imported integration, but no details are returned if it already exists.

Expected behavior

The API response should be the same data structure regardless of whether the integration was created or not. The ID is used to retrieve the integration details on import, so it is possible that the ID for an existing integration no longer falls under the same data structure. Will need to investigate further.

Screenshot

The following illustrates an integration that already exists and has been imported:

When I delete it and re-import it, I get the following:

Request and Response Data

May or may not be relevant; will be investigating further and adding details here.

API Endpoints and Schemas

POST to /api/v1/integrations

Tasks involved / Steps to Reproduce

1. Import an integration that does not exist.
2. Import works as expected.
3. Attempt to import again, no details appear for it. This is the unexpected behavior.
4. Delete the integration.
5. Import again.
6. Import works as expected.

[gashcrumb]

Ah, I see… There’s another issue for reworking the backend to “stage” the import first, when I track down the number for that we can link this one to it, as I think that’d be required to support this one.

On Wed, Oct 3, 2018, 5:50 AM Rachel Yordán [e-mail] wrote:

@gashcrumb https://github.com/gashcrumb - I know that issue is all-encompassing and relevant, but I do think it’s a separate consideration and feature if we want to let the user know that this particular integration has already been imported and show them an alert of some kind. Almost like a validation step. Or do you think we should track that from the other issue?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/syndesisio/syndesis/issues/2501#issuecomment-426576044, or mute the thread https://github.com/notifications/unsubscribe-auth/AAVdrD_7ZcIQuQZAFC0Ahr1JVQ3SD75iks5uhIhHgaJpZM4TqqJx .

[stale[bot]]

This issue has been automatically marked as stale because it has not had any activity since 90 days. It will be closed if no further activity occurs within 7 days. Thank you for your contributions!