Skip to content
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

cloudflare: update documentation to specify the unit of time-based options #2239

Merged
merged 4 commits into from
Aug 10, 2024

Conversation

kylejuliandev
Copy link
Contributor

Specify CLOUDFLARE additional configuration timeouts and intervals are in seconds

Specify CLOUDFLARE additional configuration timeouts and intervals are in seconds
@ldez
Copy link
Member

ldez commented Aug 10, 2024

<!-- THIS DOCUMENTATION IS AUTO-GENERATED. PLEASE DO NOT EDIT. -->
<!-- providers/dns/cloudflare/cloudflare.toml -->
<!-- THIS DOCUMENTATION IS AUTO-GENERATED. PLEASE DO NOT EDIT. -->

@ldez ldez changed the title Update Cloudflare DNS documentation to specify seconds in Additional Configuration options cloudflare: update documentation to specify seconds in Additional Configuration options Aug 10, 2024
@kylejuliandev
Copy link
Contributor Author

Oops! I will fix this shortly :)

Specify CLOUDFLARE additional config for intervals and timeouts are in seconds
@ldez ldez added this to the v4.18 milestone Aug 10, 2024
@ldez ldez changed the title cloudflare: update documentation to specify seconds in Additional Configuration options cloudflare: update documentation to specify the unit of time-based options Aug 10, 2024
Copy link
Member

@ldez ldez left a comment

Choose a reason for hiding this comment

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

LGTM

@ldez ldez enabled auto-merge (squash) August 10, 2024 21:15
@ldez ldez disabled auto-merge August 10, 2024 21:15
@ldez ldez enabled auto-merge (squash) August 10, 2024 21:15
@ldez ldez merged commit 24f4214 into go-acme:master Aug 10, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants