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

Warnings and errors in console #6693

Closed
11 of 15 tasks
asteriscos opened this issue May 22, 2024 · 3 comments · Fixed by #6698, #6701, #6707, #6714 or #6722
Closed
11 of 15 tasks

Warnings and errors in console #6693

asteriscos opened this issue May 22, 2024 · 3 comments · Fixed by #6698, #6701, #6707, #6714 or #6722
Assignees
Labels
level/task Task issue type/bug Bug issue

Comments

@asteriscos
Copy link
Member

asteriscos commented May 22, 2024

Wazuh Rev Browser
4.9.0 00 -

Description

The app has several warnings and some errors printed in the browser console. We need to address them.

Tasks

Evidence image
Evidence image

image

image image
  • componentWillReceiveProps has been renamed, and is not recommended for use. (It comes from the embeddables plugin)
Evidence image

image

Evidence

image
image

Evidence

image

  • EuiButtonIcon requires aria-label or aria-labelledby to be specified because icon-only buttons are screen-reader-inaccessible without them.
Evidence

image

Evidence

image

Evidence

image
image

Steps for reproduce:

  • Go to Threat intelligence > Threat Hunting and change between tabs quickly
Evidence

image

Evidence

image

Evidence

image

Evidence

image

Evidence

image

Evidence

image

Evidence

image

@asteriscos asteriscos added type/bug Bug issue level/task Task issue labels May 22, 2024
@wazuhci wazuhci moved this to Backlog in Release 4.9.0 May 22, 2024
@chantal-kelm chantal-kelm self-assigned this May 22, 2024
@chantal-kelm chantal-kelm moved this from Backlog to In progress in Release 4.9.0 May 22, 2024
@JuanGarriuz JuanGarriuz self-assigned this May 22, 2024
@chantal-kelm chantal-kelm linked a pull request May 22, 2024 that will close this issue
6 tasks
@JuanGarriuz JuanGarriuz linked a pull request May 23, 2024 that will close this issue
10 tasks
@chantal-kelm chantal-kelm linked a pull request May 23, 2024 that will close this issue
6 tasks
@JuanGarriuz JuanGarriuz linked a pull request May 24, 2024 that will close this issue
9 tasks
@yenienserrano yenienserrano self-assigned this May 28, 2024
@yenienserrano yenienserrano linked a pull request May 28, 2024 that will close this issue
8 tasks
@yenienserrano
Copy link
Member

yenienserrano commented May 28, 2024

componentWillReceiveProps has been renamed, and is not recommended for use.

I think it is better to wait for a fix from opensearch to avoid future conflicts in future updates.

image

@yenienserrano
Copy link
Member

yenienserrano commented May 29, 2024

Error: Embeddable has been destroyed

When rendered correctly in the events view it loads 1 single visualisation accordingly.

image image

But when a quick tab change is made, the dashboards of the dashboard are sent to events and cause it to break.

image image

@yenienserrano yenienserrano linked a pull request May 29, 2024 that will close this issue
6 tasks
@yenienserrano yenienserrano linked a pull request May 29, 2024 that will close this issue
6 tasks
@yenienserrano yenienserrano linked a pull request May 29, 2024 that will close this issue
6 tasks
@wazuhci wazuhci moved this from In progress to Done in Release 4.9.0 May 31, 2024
@asteriscos asteriscos linked a pull request May 31, 2024 that will close this issue
6 tasks
@chantal-kelm
Copy link
Member

It was decided to postpone this task in order to build a common solution for all components. This will be done in this issue.

Evidence image

image

image image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment