Skip to content

Conversation

@borditamas
Copy link
Collaborator

No description provided.

Copy link
Collaborator

@rbocsi rbocsi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see inline comments

The service operation **request** requires an [identity related header or certificate](../authentication_policy.md/#http) and a [ServiceOrchestrationSubscriptionRequest](../data-models/service-orchestration-subscription-request.md)
JSON encoded body.
JSON encoded body. The URI can contain an optional query parameter with the key "_trigger_" and a [Boolean](../primitives.md#boolean) value. If _trigger_ is true, then the orchestration process also will be executed.
If we add this here, the dynamic orchestration http idd should be updated as well.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems this line is a copy-paste accident.

The service operation **request** requires an [identity related header or certificate](../authentication_policy.md/#http) and a [ServiceOrchestrationSubscriptionRequest](../data-models/service-orchestration-subscription-request.md)
JSON encoded body.
JSON encoded body. The URI can contain an optional query parameter with the key "_trigger_" and a [Boolean](../primitives.md#boolean) value. If _trigger_ is true, then the orchestration process also will be executed.
If we add this here, the dynamic orchestration http idd should be updated as well.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems this line is a copy-paste accident.

Copy link
Collaborator

@rbocsi rbocsi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But please fix the two copy-paste problems. See inline comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants