Skip to content

Conversation

@kewynakshlley
Copy link
Collaborator

No description provided.

lucasfcosta
lucasfcosta previously approved these changes Sep 22, 2025
Copy link
Contributor

@lucasfcosta lucasfcosta left a comment

Choose a reason for hiding this comment

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

The logic LGTM but please see my comment about the typo and exposing the object for API keys and Domains.

* @param data The list of DomainDTO objects.
*/
public ListDomainsResponse(List<DomainDTO> data) {
public ListDomainsResponse(List<DomainDTO> data, Boolean hasMore) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we could have object: "list" exposed here too now?

@kewynakshlley kewynakshlley merged commit 1f0f739 into main Sep 22, 2025
3 checks passed
@kewynakshlley kewynakshlley deleted the feat/list-emails-and-pagination branch September 22, 2025 16:49
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.

4 participants