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

User Guide - DOM - Tutorial - Dom Viewer #4181

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ruicamacho
Copy link
Contributor

No description provided.

@ruicamacho ruicamacho requested a review from a team as a code owner January 23, 2025 14:24
@MariekeGO MariekeGO self-assigned this Jan 27, 2025
@MariekeGO
Copy link
Member

@ruicamacho Given the nature of this content, it doesn't really fit in the tutorial format. Giving a quick tour of the UI is something we typically do on a how-to page, e.g. Working with the EVS Cerebrum application. As you probably noticed yourself, the tutorial format with its strict order of steps just doesn't fit with this kind of content.
As such, I think it would be better if we changed this to a regular how-to page. This may also be the perfect chance for us to properly add the DOM Viewer solution to the Solutions section of the documentation. Right now it's only mentioned in the DataMiner Overview, but I'm not sure if that content even really belongs there. In the overview section, I think users will expect more generic bird's-eye info on the general capabilities of DataMiner, rather than the specific help page for one solution.
My recommendation would therefore be to make a DOM Viewer section where appropriate under DataMiner Solutions, and to move both the content from the page in the DataMiner Overview (or most of it) and the content of the new page in your pull request over there. Let me know if you agree or if you want to discuss this further! If you don't have much time, we can take care of everything for you.

@ruicamacho
Copy link
Contributor Author

Hey Marieke.
Thank you for the feedback.
I think I will defer to your better judgement and implement your proposed changes.
I have moved the existing "Dom Viewer (About)" and the new "Using Dom Viewer" pages from DataMiner Overview to Solutions under DevTools.
Hopefully you'll agree.
If not let me know what additional changes you wish me to do.

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