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

feat(cloud): Add custom cloud API URL configuration support #181

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from

Conversation

adamshiervani
Copy link
Contributor

@adamshiervani adamshiervani commented Feb 21, 2025

  • Implement RPC methods to set, get, and reset cloud URL
  • Modify UI to allow configuring custom cloud API URL in developer settings
  • Make a clear distinction between the different modes and their environment variables
  • Dev build only needs to know the hostname of the Cloud UI

- Implement RPC methods to set, get, and reset cloud URL
- Update cloud registration to remove hardcoded cloud API URL
- Modify UI to allow configuring custom cloud API URL in developer settings
- Remove environment-specific cloud configuration files
- Simplify cloud URL configuration in UI config
@adamshiervani adamshiervani requested a review from ym February 21, 2025 00:46
@timothystewart6
Copy link
Contributor

I just tested everything and I was able to register! Once registering though I am getting a 404 on /session. For your session calls, what is the full URL for you (with host & port) I will debug some more in the morning. Thanks again!

…ear distinction between CLOUD_API and DEVICE_API.
@adamshiervani adamshiervani requested a review from ym February 21, 2025 09:16
@adamshiervani
Copy link
Contributor Author

@timothystewart6 Fixed! Please try again

@timothystewart6
Copy link
Contributor

timothystewart6 commented Feb 21, 2025

@adamshiervani Success! The entire solution can now be (easily) developed locally. Thank you!

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