Skip to content

Conversation

mahipdeora25
Copy link
Contributor

What does this PR do?

Motivation

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

Copy link

github-actions bot commented Sep 8, 2025

⚠️ Recommendation: Add qa/skip-qa Label

This PR does not modify any files shipped with the agent.

To help streamline the release process, please consider adding the qa/skip-qa label if these changes do not require QA testing.

Copy link
Contributor

@AAraKKe AAraKKe left a comment

Choose a reason for hiding this comment

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

A suggestion you can apply and a quick question about the position of some of the elements. Other than that it looks good!

Comment on lines +1717 to +1721
"height": 24,
"is_column_break": true,
"width": 12,
"x": 0,
"y": 0
Copy link
Contributor

Choose a reason for hiding this comment

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

Just to double check, the PR says that the intention is to fix spelling errors but I have noticed that some of the elements in the dashboard have also changed position. Like this one that before was

"layout": {
              "x": 0,
              "y": 0,
              "width": 12,
              "height": 1
            }

if this is expected that's fine, just wanted to point it out in case it was unintended.

I have seen this happening in a couple of other elements as well.

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.

2 participants