New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Renamed/deprecated flexible datasources and resources (subscriptions and databases) #493
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Have a few comments on the documentation. Will approve for the sake of the build and you can bundle them in to the next PR if you prefer.
|
||
The `alert` block supports: | ||
|
||
* `name` The alert name |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* `name` The alert name | |
* `name` - The alert name |
super pedantic but think it would look better like this
The `alert` block supports: | ||
|
||
* `name` The alert name | ||
* `value` The alert value |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* `value` The alert value | |
* `value` - The alert value |
layout: "rediscloud" | ||
page_title: "Redis Cloud: rediscloud_flexible_database" | ||
description: |- | ||
Database resource in the Terraform provider Redis Cloud. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Database resource in the Terraform provider Redis Cloud. | |
Database resource in the Redis Cloud Terraform provider. |
layout: "rediscloud" | ||
page_title: "Redis Cloud: rediscloud_flexible_subscription" | ||
description: |- | ||
Flexible Subscription data source in the Terraform provider Redis Cloud. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Flexible Subscription data source in the Terraform provider Redis Cloud. | |
Flexible Subscription data source in the Redis Cloud Terraform provider. |
layout: "rediscloud" | ||
page_title: "Redis Cloud: rediscloud_flexible_subscription" | ||
description: |- | ||
Flexible Subscription resource in the Terraform provider Redis Cloud. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Flexible Subscription resource in the Terraform provider Redis Cloud. | |
Flexible Subscription resource in the Redis Cloud Terraform provider. |
data "rediscloud_payment_method" "card" { | ||
card_type = "Visa" | ||
} | ||
resource "rediscloud_subscription" "subscription-resource" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldn't the resource now be called rediscloud_flexible_subscription
in the example?
|
||
## Import | ||
|
||
`rediscloud_subscription` can be imported using the ID of the subscription, e.g. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
`rediscloud_subscription` can be imported using the ID of the subscription, e.g. | |
`rediscloud_flexible_subscription` can be imported using the ID of the subscription, e.g. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BTW, there might be other cases like this that I've missed
ecd9cf3
to
c3c27eb
Compare
No description provided.