Skip to content

APP-8173: Ability to test Viamapps locally by standing a small proxy through the CLI #5116

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

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

Conversation

NickPPC
Copy link
Member

@NickPPC NickPPC commented Jul 3, 2025

https://viam.atlassian.net/browse/APP-8173

Relies on https://github.com/viamrobotics/viam-apps/pull/57

Built on top of #5036 and replacing it

When going to localhost:8080 (if running viamapps locally) or local.app_namepsace.viamapps.com it will allow the regular flow (login + machine picking) and then it will redirect to localhost:8000/machine/<machineID>.

CLI command (viam module local-app-testing --app-url http://localhost:5173) to stand up a proxy strip the /machine/<machine ID> when serving file and add the <base href="/machine/<machine ID> > on the way back.

This gives us a full flow for testing

Screen.Recording.2025-07-03.at.3.05.09.PM.mov

@NickPPC NickPPC requested a review from a team as a code owner July 3, 2025 19:14
@NickPPC NickPPC requested review from njooma and gabegottlob July 3, 2025 19:14
@viambot viambot added the safe to test This pull request is marked safe to test from a trusted zone label Jul 3, 2025
@NickPPC NickPPC requested review from ohEmily, jr22 and bashar-515 July 3, 2025 19:16
@NickPPC NickPPC marked this pull request as draft July 3, 2025 19:19
@NickPPC NickPPC changed the title Ability to test Viamapps locally by standing a small proxy through the CLI APP-8173: Ability to test Viamapps locally by standing a small proxy through the CLI Jul 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
safe to test This pull request is marked safe to test from a trusted zone
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants