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

fix(port forward): adding tooltip for disabled port forwards #482

Merged
merged 4 commits into from
Jan 14, 2025
Merged

Conversation

stephdl
Copy link
Contributor

@stephdl stephdl commented Jan 10, 2025

Introduce a tooltip for inactive port forwards when the header contains a hyphen and add translations for the inactive status and range object compatibility.

NethServer/nethsecurity#898

image

image

image

if I revert the host range

image
image

Note:
it seems we need to save the upper unsaved change button if we want the change is effective

@stephdl stephdl force-pushed the sdl-898 branch 2 times, most recently from a2df8d7 to b4286f9 Compare January 13, 2025 10:56
gsanchietti
gsanchietti previously approved these changes Jan 13, 2025
Copy link
Member

@gsanchietti gsanchietti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a minor fix on a label.
From a user point of view, it looks good to me

andre8244
andre8244 previously approved these changes Jan 13, 2025
Co-authored-by: Giacomo Sanchietti <[email protected]>
@stephdl stephdl dismissed stale reviews from andre8244 and gsanchietti via 69fed4d January 13, 2025 15:56
Copy link
Collaborator

@andre8244 andre8244 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remember to squash and use a commit with conventional syntax :)

@Tbaile Tbaile changed the title Port forward: Add tooltip for inactive port forwards and translations fix(port forward): adding tooltip for disabled port forwards Jan 13, 2025
@stephdl stephdl merged commit 27a7e64 into main Jan 14, 2025
3 checks passed
@stephdl stephdl deleted the sdl-898 branch January 14, 2025 08:26
gsanchietti added a commit that referenced this pull request Jan 20, 2025
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