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.

Sign up for GitHub

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

Jump to bottom

[Security Solution] "Tour" under rules cannot go back to Step 1 #126084

Closed
MikePaquette opened this issue Feb 21, 2022 · 3 comments · Fixed by #128405
Closed

[Security Solution] "Tour" under rules cannot go back to Step 1 #126084

MikePaquette opened this issue Feb 21, 2022 · 3 comments · Fixed by #128405
Assignees
@banderror
Labels
8.2 candidate considered, but not committed, for 8.2 release bug Fixes for quality problems that affect the customer experience Feature:Rule Management Security Solution Detection Rule Management impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. Team:Detection Rule Management Security Detection Rule Management Team Team:Detections and Resp Security Detection Response Team Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. v8.1.2

Comments

@MikePaquette
Copy link

Kibana version: 8.1.0 BC1

Elasticsearch version: 8.1.0 BC1

Server OS version: Elastic Cloud ESS default

Browser version: Google Chrome Version 98.0.4758.102 (Official Build) (x86_64)

Browser OS version: macOS Monterey Version 12.2.1 (21D62)

Original install method (e.g. download page, yum, from source, etc.): Elastic Cloud ESS GCP Europe-West-1

Describe the bug:
When first entering the rules page, I see a "Tour" to help me understand the page. The Tour seemed to appear at Step 2, and did not allow me to go back to Step 1.

Steps to reproduce:

  1. Create a new deployment
  2. Visit Rule Page
  3. Load prebuilt detection rules
  4. Look at Rule page
  5. See the "Tour" helper panel at Step 2

Expected behavior:
Tour should start at Step 1 and be able to go back to Step 2 if I am on Step 2.

Screenshots (if relevant):

image

Errors in browser console (if relevant): None

Provide logs and/or server output (if relevant): N/A

Any additional context: None
@MikePaquette MikePaquette added bug Fixes for quality problems that affect the customer experience impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Feature:Rule Management Security Solution Detection Rule Management labels Feb 21, 2022
@MadameSheema MadameSheema added Team:Detection Rule Management Security Detection Rule Management Team triage_needed labels Feb 21, 2022
@MindyRS MindyRS added the Team:Detections and Resp Security Detection Response Team label Feb 23, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-detections-response (Team:Detections and Resp)

@peluja1012
Copy link
Contributor

I created a brand new 8.1.0 deployment in Cloud and I see Step 1 on initial load. See screenshot below.

image

I did notice that after the first step is acknowledged by the user, there is no way to navigate to the previous step. However, given that Step 1 does appear on initial load and given the fact that this is only a Tour feature and our enhancements are also documented in our docs, I hesitate classifying it as impact: high. @jethr0null Let me know if you disagree.

@peluja1012 peluja1012 added impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. and removed impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. triage_needed labels Mar 3, 2022
@banderror
Copy link
Contributor

I added this issue to the 8.2 checklist - we may consider addressing it in 8.1.1.
However, just want to point out that the Tour UI will be removed in 8.2.0: #125504

@banderror banderror removed their assignment Mar 7, 2022
@banderror banderror added the 8.2 candidate considered, but not committed, for 8.2 release label Mar 7, 2022
@banderror banderror self-assigned this Mar 10, 2022
@banderror banderror mentioned this issue Mar 23, 2022
Merged
@banderror banderror linked a pull request Mar 23, 2022 that will close this issue
[Security Solution][Detections] Adds a back button to the Tour UI on the Rule Management page #128405
Merged
banderror added a commit that referenced this issue Mar 24, 2022
@banderror
[Security Solution][Detections] Adds a back button to the Tour UI on …
61f805b 
…the Rule Management page (#128405)

**Ticket:** #126084

## Summary

This PR adds a back button to each of the two Tour steps. Users will be able to freely navigate between the steps.

## Before

![](https://user-images.githubusercontent.com/92328789/152391079-57a12b97-a273-4852-95e6-222680d09464.png)

![](https://user-images.githubusercontent.com/92328789/152391090-87cd436a-eb74-4358-b699-53c3e79fdb48.png)

## After

![](https://puu.sh/IQoYO/d213b38e5d.png)

![](https://puu.sh/IQoYT/bd1c055be3.png)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@banderror banderror

Labels
8.2 candidate considered, but not committed, for 8.2 release bug Fixes for quality problems that affect the customer experience Feature:Rule Management Security Solution Detection Rule Management impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. Team:Detection Rule Management Security Detection Rule Management Team Team:Detections and Resp Security Detection Response Team Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. v8.1.2
Projects
None yet
Milestone
No milestone
6 participants
@peluja1012 @MindyRS @banderror @elasticmachine @MadameSheema @MikePaquette