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

Update EmailForward fields #175

Closed
wants to merge 2 commits into from
Closed

Conversation

ggalmazor
Copy link
Contributor

See dnsimple/dnsimple-developer#315 for reference

The Go API client already defines the new fields and deprecates the old ones. However, it defines a bogus AliasName that must not be used.

In this PR:

  • Deprecate AliasName in EmailForward

Also fix deprecation comments to align them with out current API
@ggalmazor ggalmazor self-assigned this May 17, 2024
@ggalmazor ggalmazor added the enhancement New feature, enhancement or code changes, not related to defects label May 17, 2024
@ggalmazor
Copy link
Contributor Author

Closing. I was mistaken with the AliasName field in EmailForward. No changes are actually required for dnsimple/dnsimple-developer#315 in this client

@ggalmazor ggalmazor closed this May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature, enhancement or code changes, not related to defects
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant