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

Add/Update help icon linking to docs #2720 #2725

Conversation

FroggyFlox
Copy link
Member

Fixes #2720

In the configuration modals for our Services, some include links to their respective docs while other do not.

This pull requests harmonizes these discrepancies by adding the same help icon linking to the Service's section in our docs.

Also includes:

  • minor update of Tailscale service configuration header so that it matches the other services
  • update copyright statements

The configuration modals for all affected services were verified:

  • to include the new/updated HELP icon
  • to link to the appropriate section of our docs

In the configuration modals for our Services, some include links to
their respective docs while other do not.

This commit harmonizes these discrepancies by adding the same help icon
linking to the Service's section in our docs.
Also includes:
  - minor update of Tailscale service configuration header so that it
    matches the other services
  - update copyright statements
@phillxnet
Copy link
Member

Linking to related, and now merged/published docs PR:
"Match list of services in docs to Rockstor UI ...": rockstor/rockstor-doc#462

Copy link
Member

@phillxnet phillxnet left a comment

Choose a reason for hiding this comment

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

@FroggyFlox A functional test via an rpm build. Thanks for this excellent use experience enhancement. And for the linked PR to arrange our docs end appropriately. Nice.

Successfully built an rpm against current testing branch.

Confirming that all Web-UI System -> Service spanner-icon-invoked config dialogs link to respective doc entries successfully.

pr2725-service-doc-help-links

@phillxnet
Copy link
Member

@FroggyFlox I'll go ahead an merge this one then, as it's been ready for some time now I see. Otherwise we may end up encountering some unwelcome merge conflicts.

@phillxnet phillxnet merged commit 5fa419e into rockstor:testing Nov 28, 2023
@phillxnet phillxnet added this to the 5.1.X-X Stable release milestone Nov 28, 2023
@FroggyFlox FroggyFlox deleted the 2720_Add-Update-HELP-icons-links-to-docs branch December 3, 2023 17:14
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.

2 participants