Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

Invoking metadata endpoints for non dynamic actions #4914

Closed
zregvart opened this issue Mar 15, 2019 · 4 comments · Fixed by #4917
Closed

Invoking metadata endpoints for non dynamic actions #4914

zregvart opened this issue Mar 15, 2019 · 4 comments · Fixed by #4917
Assignees
Labels
cat/bug A bug which needs fixing group/ui User interface SPA, talking to the REST backend ignore/qe For issues that aren't supposed to be verified by QE notif/triage The issue needs triage. Applied automatically to all new issues. prio/p2 The priority of a bug. p1 means low

Comments

@zregvart
Copy link
Member

This is a...


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

The problem

In API provider integrations I noticed that we invoke /api/v1/connections/api-provider/actions/io.syndesis:api-provider-start and /api/v1/connections/api-provider/actions/io.syndesis:api-provider-end endpoints -- the action metadata. For actions that are not tagged with dynamic. The generated integration already contains the shapes based on the OpenAPI specification and invoking the action metadata endpoint for them will yield any data shape as that's what the connector descriptor defines.

Expected behavior

Invoking action metadata endpoint only for actions tagged with dynamic.

Tasks involved / Steps to Reproduce

  1. Create API provider integration based on the TODO OpenAPI
  2. While implementing one of the flows notice the invocations of /api/v1/connections/api-provider/actions/io.syndesis:api-provider-start and /api/v1/connections/api-provider/actions/io.syndesis:api-provider-end endpoints
@zregvart zregvart added cat/bug A bug which needs fixing group/ui User interface SPA, talking to the REST backend prio/p2 The priority of a bug. p1 means low labels Mar 15, 2019
@pure-bot pure-bot bot added the notif/triage The issue needs triage. Applied automatically to all new issues. label Mar 15, 2019
@gashcrumb gashcrumb self-assigned this Mar 15, 2019
@gashcrumb
Copy link
Contributor

Should be easy enough to fix, those are done here.

@mastepan
Copy link

@gashcrumb please keep this open for the QE be able to track and verify. Thanks

@mastepan mastepan reopened this Jun 13, 2019
@mastepan
Copy link

mastepan commented Aug 7, 2019

@gashcrumb I got back to this. Is there anything that QE should check? Or can we add ignore/QE label?

@tplevko
Copy link
Contributor

tplevko commented Aug 9, 2019

This is related to the old UI, so not really relevant anymore. Closing with QE_ignore

@tplevko tplevko closed this as completed Aug 9, 2019
@tplevko tplevko added the ignore/qe For issues that aren't supposed to be verified by QE label Aug 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cat/bug A bug which needs fixing group/ui User interface SPA, talking to the REST backend ignore/qe For issues that aren't supposed to be verified by QE notif/triage The issue needs triage. Applied automatically to all new issues. prio/p2 The priority of a bug. p1 means low
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants