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

Adds advanced settings descriptions #6928

Merged
merged 62 commits into from
May 13, 2024
Merged

Adds advanced settings descriptions #6928

merged 62 commits into from
May 13, 2024

Conversation

vagimeli
Copy link
Contributor

@vagimeli vagimeli commented Apr 9, 2024

Description

Adds advanced settings descriptions.

Issues Resolved

Closes #6344
Closes #7051

Checklist

  • By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
    For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@vagimeli vagimeli added 3 - Tech review PR: Tech review in progress backport 2.13 PR: Backport label for 2.13 labels Apr 9, 2024
@vagimeli vagimeli self-assigned this Apr 9, 2024
@vagimeli
Copy link
Contributor Author

vagimeli commented Apr 9, 2024

@smacrakis Are you available to provide a tech review of the added advanced settings content? If not, will you provide me with a developer point of contact? This PR addresses your issue #6344 regarding advanced settings descriptions. Thank you, Melissa

@vagimeli vagimeli added v2.14.0 and removed backport 2.13 PR: Backport label for 2.13 labels May 3, 2024
@vagimeli
Copy link
Contributor Author

vagimeli commented May 3, 2024

@ananzh I've updated the quickstart to include the new feature for switching appearance themes from the home page. We have this documentation on the schedule to release with 2.14. Please review at your earliest availability. Thank you.

@hdhalter hdhalter added the release-notes PR: Include this PR in the automated release notes label May 6, 2024
@abbyhu2000 abbyhu2000 self-assigned this May 6, 2024
_dashboards/quickstart.md Outdated Show resolved Hide resolved
_dashboards/quickstart.md Outdated Show resolved Hide resolved
Copy link
Member

Choose a reason for hiding this comment

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

duplicate icon file with images/icons/theme-icon.png?

_dashboards/management/advanced-settings.md Outdated Show resolved Hide resolved
_dashboards/management/advanced-settings.md Outdated Show resolved Hide resolved
_dashboards/management/advanced-settings.md Outdated Show resolved Hide resolved
_dashboards/management/advanced-settings.md Outdated Show resolved Hide resolved
_dashboards/management/advanced-settings.md Outdated Show resolved Hide resolved
_dashboards/management/advanced-settings.md Outdated Show resolved Hide resolved
@hdhalter
Copy link
Contributor

hdhalter commented May 8, 2024

Thanks, @abbyhu2000! I'll move this to doc review and we'll let you know if we have follow up questions.

@hdhalter hdhalter added 4 - Doc review PR: Doc review in progress and removed 3 - Tech review PR: Tech review in progress labels May 8, 2024
vagimeli and others added 17 commits May 10, 2024 11:56
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
@vagimeli
Copy link
Contributor Author

vagimeli commented May 10, 2024

@natebower Thank you for the thorough review. I've accepted your suggestions and addressed feedback. Please approval at your availability.

Copy link
Collaborator

@natebower natebower left a comment

Choose a reason for hiding this comment

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

LGTM

@vagimeli vagimeli merged commit 8d23155 into main May 13, 2024
7 checks passed
@vagimeli vagimeli deleted the add-settings-descriptions branch May 13, 2024 15:59
@vagimeli vagimeli removed the 5 - Editorial review PR: Editorial review in progress label May 13, 2024
@hdhalter hdhalter added 3 - Done Issue is done/complete labels May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Done Issue is done/complete release-notes PR: Include this PR in the automated release notes v2.14.0
Projects
None yet
5 participants