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

Restored legacy PagedResponse structs and added deprecation notice #539

Conversation

ezilber-akamai
Copy link
Contributor

📝 Description

Restored legacy PagedResponse structs and added deprecation notice

@ezilber-akamai ezilber-akamai requested a review from a team as a code owner July 2, 2024 17:47
@ezilber-akamai ezilber-akamai requested review from jriddle-linode and ykim-akamai and removed request for a team July 2, 2024 17:47
@zliang-akamai
Copy link
Member

Just a thought, since they are all in the root package linodego, would it be better to put every deprecated structs in a single file?

@ezilber-akamai
Copy link
Contributor Author

Just a thought, since they are all in the root package linodego, would it be better to put every deprecated structs in a single file?

Good idea! It would make it much simpler to phase them out when that eventually happens.

@ezilber-akamai ezilber-akamai force-pushed the TPT-3026-restore_paged_response_structs branch from c8fa872 to 6d16efe Compare July 2, 2024 19:13
Copy link
Member

@zliang-akamai zliang-akamai left a comment

Choose a reason for hiding this comment

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

Looks good to me! Nice work!

@@ -0,0 +1,55 @@
package linodego

// Deprecated: AccountAvailabilityPagedResponse exists for historical compatibility and should not be used.
Copy link
Contributor

Choose a reason for hiding this comment

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

optional: We can do something like this to make this a bit tidier:

type (
	// Deprecated: AccountAvailabilityPagedResponse exists for historical compatibility and should not be used.
	AccountAvailabilityPagedResponse legacyPagedResponse[AccountAvailability]

	// Deprecated: AccountBetasPagedResponse exists for historical compatibility and should not be used.
	AccountBetasPagedResponse legacyPagedResponse[AccountBetaProgram]
	
	// ...
)

Copy link
Contributor

@lgarber-akamai lgarber-akamai left a comment

Choose a reason for hiding this comment

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

Looks great, nice work!

@ezilber-akamai ezilber-akamai merged commit a9a7dcf into linode:main Jul 3, 2024
4 checks passed
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