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: add settings for public url of the spoolman frontend #1602

Closed
wants to merge 1 commit into from

Conversation

Roguyt
Copy link

@Roguyt Roguyt commented Oct 10, 2023

Description

This allow to specify the real IP or domain of the spoolman client which is used in the Mainsail UI to open it through the two buttons "Open Spool Manager"

This resolves an issue I have since my spoolman instance is set as 127.0.0.1 for moonraker but this isn't obviously the good link to access the Web UI.

Mobile & Desktop Screenshots/Recordings

image

[optional] Are there any post-deployment tasks we need to perform?

No.

@github-actions
Copy link
Contributor

Language file analysis report:

File Missing Keys Unused Keys
en.json 0 0

@github-actions
Copy link
Contributor

Language file analysis report:

File Missing Keys Unused Keys
en.json 0 0

@meteyou
Copy link
Member

meteyou commented Oct 11, 2023

@Roguyt Thx for your PR! Would it be also possible to just replace 127.0.0.1/localhost with the current moonraker connection host?

@meteyou meteyou added the ❔ User Input Further information is requested label Oct 22, 2023
Copy link
Contributor

Ahoi!

It looks like there hasn't been any recent updates on
this pull request. If you created this pull request and
no longer consider it to get merged, then please login
to github and close it. Otherwise, if there is no further
activity on this pull request, it will be automatically
closed within the next 7 days.

Fair wind and a following sea!
~ Your friendly MainsailGithubBot

PS: I'm just an automated script, not a real sailor.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💤 Stale ❔ User Input Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants