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_dns: Rename account_api_token parameter #62707

Merged
merged 1 commit into from
Sep 21, 2019

Conversation

andreaso
Copy link
Contributor

SUMMARY

Name change triggered by the new parameter api_token (8cabf1c) being
a bit too similar to the old name account_api_token. At the same
time account_api_key is actually a better name for the original
parameter, in it mapping closer to the terminology used by Cloudflare.

Keeping account_api_token as an alias for backward compatibility.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

cloudflare_dns

Name change triggered by the new parameter _api_token_ (8cabf1c) being
a bit too similar to the old name _account_api_token_. At the same
time _account_api_key_ is actually a better name for the original
parameter, in it mapping closer to the terminology used by Cloudflare.

Keeping _account_api_token_ as an alias for backward compatibility.
Copy link
Contributor

@resmo resmo left a comment

Choose a reason for hiding this comment

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

shipit

@resmo resmo merged commit fc6a37f into ansible:devel Sep 21, 2019
anas-shami pushed a commit to anas-shami/ansible that referenced this pull request Sep 23, 2019
Name change triggered by the new parameter _api_token_ (8cabf1c) being
a bit too similar to the old name _account_api_token_. At the same
time _account_api_key_ is actually a better name for the original
parameter, in it mapping closer to the terminology used by Cloudflare.

Keeping _account_api_token_ as an alias for backward compatibility.
@ansible ansible locked and limited conversation to collaborators Oct 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants