dockge icon indicating copy to clipboard operation
dockge copied to clipboard

Port Editor does not update Clickable Ports, but YAML editor does

Open mrstux opened this issue 1 year ago • 2 comments

⚠️ Please verify that this bug has NOT been reported before.

  • [X] I checked and didn't find similar issue

🛡️ Security Policy

Description

Dockge has clickable links to exposed ports

Screenshot 2024-11-28 at 3 26 05 PM

When using the Dockge port editor to add ports, eg:

Screenshot 2024-11-28 at 3 27 19 PM

The clickable ports are not updated when clicking save.

eg:

Screenshot 2024-11-28 at 3 26 05 PM

If instead, the YAML is edited, then they are Screenshot 2024-11-28 at 3 28 26 PM

Screenshot 2024-11-28 at 3 28 38 PM

👟 Reproduction steps

Using Dockge 1.4.2

Click "+ Compose", name the default stack, then deploy it

Then click "Edit"

Then in the stack editor, click Edit again

Screenshot 2024-11-28 at 3 32 10 PM

Change the port. Then click save.

The port will not change, see below, YAML is updated, clickable exposed port is not.

Screenshot 2024-11-28 at 3 33 16 PM

Click edit again, then change the port in the YAML... then click save. The port will change,

Screenshot 2024-11-28 at 3 33 42 PM

👀 Expected behavior

Changes to ports in the port edit view should be reflected by the UI

😓 Actual Behavior

Changes to ports in the port edit view are not reflected by the UI

Dockge Version

1.4.2

💻 Operating System and Arch

TrueNAS Scale 24.10.0.2

🌐 Browser

Safari Version 17.6 (19618.3.11.11.5)

🐋 Docker Version

Docker version 27.1.1, build 6312585

🟩 NodeJS Version

No response

📝 Relevant log output

No response

mrstux avatar Nov 28 '24 04:11 mrstux

You need to click on 'Deploy' to update the Compose and not on 'Save'. By clicking on 'Deploy', your ports will be updated

cyril59310 avatar Nov 28 '24 10:11 cyril59310

Yes, the ports are updated in the yaml, but they are not updated in the UI, even though the application is launched with the correct ports when you use “deploy”

This appears to be a UI update issue.

mrstux avatar Nov 29 '24 00:11 mrstux