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

Incorrect tailscale and usb web interface instructions #57

Closed
rM-self-serve opened this issue Mar 11, 2024 · 5 comments · Fixed by #69
Closed

Incorrect tailscale and usb web interface instructions #57

rM-self-serve opened this issue Mar 11, 2024 · 5 comments · Fixed by #69

Comments

@rM-self-serve
Copy link
Contributor

I submitted #49 after user confirmation, but upon configuring tailscale myself I have found that tailscale can only share services running on localhost.

I have proposed a package in toltec that will allow the web interface to be accessed over tailscale. Upon that package being merged, the guide can be updated to include/point to the new instructions.

In the meantime it may be advisable to revert #49.

@Eeems
Copy link
Contributor

Eeems commented Mar 11, 2024

In the meantime it may be advisable to revert #49.

As this would remove the anchor that people may be linking to, this would not be the correct solution for the site. Updated content with a workaround to get this to work, or explaining that it's not currently possible would need to be added, or the anchor added to another place on the page.

@rM-self-serve
Copy link
Contributor Author

Would it be advisable to wait for toltec-dev/toltec#840 or shall I open a pull request here stating that the specific section/anchor is under maintenance?

@Eeems
Copy link
Contributor

Eeems commented Mar 12, 2024

Would it be advisable to wait for toltec-dev/toltec#840 or shall I open a pull request here stating that the specific section/anchor is under maintenance?

Either works for me. It'll be a little while before toltec-dev/toltec#840 gets to stable.

@Eeems
Copy link
Contributor

Eeems commented Jun 8, 2024

Since the new package is in stable, are you planning on opening a pr to update this?

@rM-self-serve
Copy link
Contributor Author

Yeah I will have some time soon!

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 a pull request may close this issue.

2 participants