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

Remove mentions of the "Advanced sorting" toggle #2960

Merged
merged 1 commit into from
Feb 13, 2023

Conversation

xcrzx
Copy link
Contributor

@xcrzx xcrzx commented Feb 8, 2023

Resolves: #2947

Summary

The advanced sorting toggle has been removed from the rules management tables; see elastic/kibana#138907. Now, users can sort by any table column without that toggle.

@xcrzx xcrzx self-assigned this Feb 8, 2023
@github-actions
Copy link

github-actions bot commented Feb 8, 2023

Documentation previews:

@mergify
Copy link
Contributor

mergify bot commented Feb 8, 2023

This pull request does not have a backport label. Could you fix it @xcrzx? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • v7.x is the label to automatically backport to the 7.x branch.
  • v7./d./d is the label to automatically backport to the 7./d branch. /d is the digit

NOTE: backport-skip has been added to this pull request.

Copy link
Contributor

@nastasha-solomon nastasha-solomon left a comment

Choose a reason for hiding this comment

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

Really appreciate you taking care of this @xcrzx ! I left two minor suggestions for your consideration. Also, the all-rules.png image needs to be refreshed, but we can probably take care of that if needed. The image shows up in the Create a detection rule page and the Manage detection rules page.

docs/detections/rules-ui-manage.asciidoc Outdated Show resolved Hide resolved
docs/detections/rules-ui-monitor.asciidoc Show resolved Hide resolved
@nastasha-solomon nastasha-solomon added the Team: Threat Hunting Formerly Data Visibility label Feb 8, 2023
@xcrzx xcrzx force-pushed the remove-advanced-sorting branch from b3681c6 to cd2401f Compare February 10, 2023 13:15
@xcrzx
Copy link
Contributor Author

xcrzx commented Feb 10, 2023

Also, the all-rules.png image needs to be refreshed

Good catch! Updated the image as well

@xcrzx xcrzx merged commit 3c37bc6 into elastic:main Feb 13, 2023
mergify bot pushed a commit that referenced this pull request Feb 13, 2023
jmikell821 pushed a commit that referenced this pull request Feb 14, 2023
(cherry picked from commit 3c37bc6)

Co-authored-by: Dmitrii Shevchenko <[email protected]>
@joepeeples joepeeples mentioned this pull request Mar 22, 2023
25 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DOCS] Remove mentions of the "Advanced sorting" toggle from the docs
2 participants