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

chore(internal): adds support for resource configuration #996

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

snorrees
Copy link

@snorrees snorrees commented Jan 28, 2025

Temp branch to explore resource targets in the api.

Current tagged release: 6.27.2-resources.1

See added unit-tests tests for what I've tested.

Caveats

Current impl takes experimental_resource or projectId resource.r<esourceType>.<resourceId> (which then sets experimental_resource config)

The client then is forced into useProjectHostname: false

Then all requests get // prepended (urls are changed)

This is probably not what we want for client.request that currently use useProjectHostname.

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.

1 participant