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

[doc] Extend documentation for detection status #3038

Merged

Conversation

csordasmarton
Copy link
Contributor

No description provided.

@csordasmarton csordasmarton added GUI 🎨 documentation 📖 Changes to documentation. labels Nov 20, 2020
@csordasmarton csordasmarton added this to the release 6.16.0 milestone Nov 20, 2020
gyorb
gyorb previously requested changes Nov 23, 2020
![Detection status change](images/detection_status_flow_chart.png)

*Notes*:
- The detection status of a report is connected to the report id. It means that
Copy link
Contributor

Choose a reason for hiding this comment

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

I think highlighting these sentences by moving them up after the first line The detection status is the state of a bug report in a run. ... would probably give more visibility.

Maybe one section about the difference between the detection status and the review status would also clarify what and when they are used or should be used. What do you think?

@bruntib
Copy link
Contributor

bruntib commented Nov 23, 2020

The image has been uploaded under vue-cli directory, but no other .vue files changed. Is this going to appear in the GUI?

@csordasmarton csordasmarton force-pushed the extend_doc_for_detection_status branch from 047c9bd to 1712704 Compare December 2, 2020 13:10
@csordasmarton csordasmarton force-pushed the extend_doc_for_detection_status branch from 1712704 to b832fd8 Compare December 2, 2020 13:11
@csordasmarton csordasmarton dismissed gyorb’s stale review December 14, 2020 07:55

The review comments are fixed.

@csordasmarton csordasmarton merged commit da26c1d into Ericsson:master Dec 14, 2020
@csordasmarton csordasmarton deleted the extend_doc_for_detection_status branch March 10, 2021 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation 📖 Changes to documentation. GUI 🎨
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants