Skip to content

Conversation

@Integralist
Copy link
Collaborator

@Integralist Integralist commented Aug 26, 2025

User Impact

There is no user impact as the change is to an internal function logic and not the exposed interface.

As this is not a public-facing change, I'm skipping adding a new entry to the changelog.

@Integralist Integralist added the Skip-Changelog Do not check for changelog diff label Aug 26, 2025
@Integralist Integralist marked this pull request as ready for review September 2, 2025 16:54
@Integralist Integralist requested a review from a team as a code owner September 2, 2025 16:54
Copy link
Member

@philippschulte philippschulte left a comment

Choose a reason for hiding this comment

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

Thank you!

@Integralist
Copy link
Collaborator Author

@tundal45 I'll wait for you to confirm this is safe to merge

@anthony-gomez-fastly
Copy link
Contributor

@Integralist @tundal45 what's the status on this?

@Integralist
Copy link
Collaborator Author

Integralist commented Sep 17, 2025

@anthony-gomez-fastly @tundal45 the updated API paths look to be public now https://www.fastly.com/documentation/reference/api/domain-management/domains/

EDIT: Oh, actually no, the links in this PR are for */tools/* not domains.

@philippschulte
Copy link
Member

@Integralist is this ready to be merged? Thanks!

@Integralist
Copy link
Collaborator Author

@philippschulte I see the docs are now live https://www.fastly.com/documentation/reference/api/domain-management/domain-research/ so I would say "yes", but maybe just get a quick confirmation from @case-fastly first.

@case-fastly
Copy link

Ship it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip-Changelog Do not check for changelog diff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants