Skip to content

[SDK-3336] - exchange Parameter for AmqpExternalTarget struct#24

Merged
graham-russell merged 1 commit intomainfrom
feature/SDK-3336-amqpexternal-rule-exchange-param
Dec 13, 2023
Merged

[SDK-3336] - exchange Parameter for AmqpExternalTarget struct#24
graham-russell merged 1 commit intomainfrom
feature/SDK-3336-amqpexternal-rule-exchange-param

Conversation

@graham-russell
Copy link
Copy Markdown
Contributor

@graham-russell graham-russell commented Dec 11, 2023

Add the exchange parameter to the AmqpExternalTarget struct. This has been implemented in the Control API and will be exposed in the Ably Terraform provider.

Infra ticket is INF-3251

@graham-russell graham-russell changed the title [SDK-3336] [SDK-3336] - Exchange Parameter for AmqpExternalTarget struct Dec 11, 2023
@graham-russell graham-russell force-pushed the feature/SDK-3336-amqpexternal-rule-exchange-param branch from 392194a to 5548c85 Compare December 11, 2023 15:55
@graham-russell graham-russell changed the title [SDK-3336] - Exchange Parameter for AmqpExternalTarget struct [SDK-3336] - exchange Parameter for AmqpExternalTarget struct Dec 11, 2023
@graham-russell graham-russell force-pushed the feature/SDK-3336-amqpexternal-rule-exchange-param branch from 5548c85 to 4db308f Compare December 11, 2023 15:56
@graham-russell graham-russell force-pushed the feature/SDK-3336-amqpexternal-rule-exchange-param branch from 4db308f to ecc3a13 Compare December 11, 2023 16:17
Copy link
Copy Markdown
Contributor

@surminus surminus left a comment

Choose a reason for hiding this comment

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

Trailing quote but otherwise seems ok

Add the `exchange` parameter to the `AmqpExternalTarget` struct. This has been implemented in the Control API and will be exposed in the Ably Terraform provider.
@graham-russell graham-russell force-pushed the feature/SDK-3336-amqpexternal-rule-exchange-param branch from ecc3a13 to 164517f Compare December 11, 2023 16:20
Copy link
Copy Markdown
Member

@owenpearson owenpearson left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@graham-russell graham-russell merged commit a41bf3d into main Dec 13, 2023
@graham-russell graham-russell deleted the feature/SDK-3336-amqpexternal-rule-exchange-param branch December 13, 2023 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants