-
Notifications
You must be signed in to change notification settings - Fork 24
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
Remove premium_price attribute from registrar order responses #67
Conversation
This is a parameter allowed for register/transfer/renew.
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.
There's some inline documentation that appears incorrect and is confusing.
dnsimple/registrar.go
Outdated
@@ -122,6 +121,9 @@ type DomainRegisterRequest struct { | |||
// Set to true to enable the auto-renewal of the domain. | |||
// Default to true. | |||
EnableAutoRenewal bool `json:"auto_renew,omitempty"` | |||
// Required as confirmation of the price, only if the domain is premium. | |||
// Default to true. |
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.
I don't understand why this says "Default to true" when this is a string 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.
It is a mistake.
dnsimple/registrar.go
Outdated
@@ -175,6 +176,9 @@ type DomainTransferRequest struct { | |||
// Set to true to enable the auto-renewal of the domain. | |||
// Default to true. | |||
EnableAutoRenewal bool `json:"auto_renew,omitempty"` | |||
// Required as confirmation of the price, only if the domain is premium. | |||
// Default to true. |
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.
I don't understand why this says "Default to true" when this is a string value.
dnsimple/registrar.go
Outdated
@@ -239,6 +242,9 @@ type domainRenewalResponse struct { | |||
type DomainRenewRequest struct { | |||
// The number of years | |||
Period int `json:"period"` | |||
// Required as confirmation of the price, only if the domain is premium. | |||
// Default to true. |
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.
I don't understand why this says "Default to true" when this is a string value.
I fixed it @aeden |
See dnsimple/dnsimple-developer#155