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

Add quick settings view #1107

Merged
merged 21 commits into from
Sep 1, 2021
Merged

Conversation

frankeros
Copy link
Contributor

@frankeros frankeros commented Sep 1, 2021

Hi team, this resolves:

  • Improve the way that the selected API, index, and source type is displayed
  • Add quick settings to change the related values easily

image

image

Closes #1101

@frankeros frankeros changed the base branch from master to 4.1-8.1 September 1, 2021 18:20
Copy link
Contributor

@matiasmoreno876 matiasmoreno876 left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@gabiwassan gabiwassan left a comment

Choose a reason for hiding this comment

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

LGTM

@frankeros frankeros merged commit 531c02d into 4.1-8.1 Sep 1, 2021
@frankeros frankeros deleted the feat/1101-add-sourcetype-selector branch September 1, 2021 18:33
Copy link
Member

@Machi3mfl Machi3mfl left a comment

Choose a reason for hiding this comment

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

CR ✅
LGTM!

frankeros added a commit that referenced this pull request Sep 14, 2021
* Enhance/show-agents-node-name (#1103)

* Add agent's cluster node name on agent's details

* Add new column to Agents overview table to show their cluster node name.

 - New column's visibility can be toggled as any other column.
 - Found out that there was already some code to display the nodes, but the object holding the data (clusterInfo) returns null. Most probably some change on the API broke this and no one noticed until now.

* Fix broken request to fecth cluster info

Agents overview controller launched an API request to detect whether Wazuh works on cluster mode or not.
It looks like the API endpoints have changed and the requested endpoints does not provide such information
anymore. This information is used to generate the search filters for the table, so the broken request resulted
on the filtering by node_name tag not showing on the search-bar filters.

- Added a new route state resolver to 'agents' view in order to fetch cluster information ('/cluster/status/).
	modified:   SplunkAppForWazuh/appserver/static/js/config/routes/agents-states.js
	modified:   SplunkAppForWazuh/appserver/static/js/controllers/agents/agents/agentsCtrl.js

* fix: table in reports show different columns than UI

* doc: add changelog

Co-authored-by: Franco Charriol <francocharriol@gmail.com>

* Take sourcetype filter for internal queries from inputs.conf (#1104)

* Added new service sourceTypeStorageService.js

* feat: replace all source type in queries with the selected value

* doc: add changelog

* Added try catch strategy sourceTypeStorageService.js

* Add quick settings view (#1107)

* feat: add sourcetype as filter

* feat: add selectAPI on wz-menu.js

* feat: add dropdown constructor

* fix: remove this

* fix: get apiList

* Added modal wazuh api selector

* fix: added index and source type selector on menu

* fix: missing value

* Daft sourcetype and index selectors - buggy

* fixed update emitter

* fix: remove listeners

* fix: remove listeners

* Added Index and SourceType selectors

* fix: reload page on change

* Fixed onchange reload

* Fixed unregister to stopListening

* fix: reload refresh sourcetype

* doc: add changelog

* fix: remove comment line

Co-authored-by: Federico Rodriguez <federico.rodriguez@wazuh.com>

* fix: quick selector throw error

* fix: improve exception logging

* fix: add missing params

Co-authored-by: Álex <alex-r-b@hotmail.com>
Co-authored-by: Matias Ezequiel Moreno <49887871+matiasmoreno876@users.noreply.github.com>
Co-authored-by: Federico Rodriguez <federico.rodriguez@wazuh.com>
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.

Take sourcetype filter for internal queries from inputs.conf
5 participants