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
Deprecate and remove premium_price attribute from registrar order responses #155
Conversation
It is going to be removed.
I'm actually torn about this commit. In theory we will still return it as nil, hence we may want to leave it here. However, we'd rather not expose it.
Going to update the clients once this PR is approved. |
It contains a DomainRenewal and not a Domain. This is connected to dnsimple/dnsimple-developer#155 as I reviewed all models as part of the change.
@@ -4112,7 +4100,7 @@ components: | |||
properties: | |||
period: | |||
type: integer | |||
preimum_price: | |||
premium_price: |
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.
Isn't premium_price supposed to be removed?
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.
Nope, this is the parameter. Not the response field.
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.
Ah, ok. 👍
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 see one instance of premium_price is retained... is that intentional?
It contains a DomainRenewal and not a Domain. This is connected to dnsimple/dnsimple-developer#155 as I reviewed all models as part of the change.
This is inconsistent with most of our code, and if we will ever return a price it will be returned for every order.
Clients
dnsimple/dnsimple-node(not required)