Skip to content

Conversation

yec-akamai
Copy link
Contributor

No description provided.

## 📝 Description

[IPv6 range
view](https://www.linode.com/docs/api/networking/#ipv6-range-view__responses)
and [IPv6 range
list](https://www.linode.com/docs/api/networking/#ipv6-ranges-list__responses)
actually share different response structures. The current `IPv6Range`
object has been built based on the structure of IPv6 range list, so
fields `linodes` and `is_bgp` are missing. To quickly fix it, I think
simply adding these two fields to the `IPv6Range` object properties
makes sense without introducing any breaking change. Otherwise we may
consider create two separated objects for returning a single object and
list, which is probably confusing for customers to use and more
complicated for us to maintain. Any different idea is appreciated.

## ✔️ How to Test

`tox`
@yec-akamai yec-akamai requested a review from a team as a code owner August 21, 2023 14:41
@zliang-akamai zliang-akamai requested a review from a team August 21, 2023 14:45
@yec-akamai yec-akamai merged commit 4301728 into main Aug 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants