Skip to content

Add UUID field to APIs table #7412

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

Merged

Conversation

gonzaarancibia
Copy link
Member

@gonzaarancibia gonzaarancibia commented May 5, 2025

Description

This pull request adds the UUID field to the Server API's table

Issues Resolved

#7387

Evidence

Screen.Recording.2025-05-06.at.12.56.19.PM.mov

Test

  1. Navigate to the Dashboard Managenment > Server APIs section in the Wazuh dashboard
  2. Verify that the UUID field appears correctly in the Server API table
  3. Click on an API ID to check that the popover displays the UUID information
  4. Test the copy to clipboard functionality by clicking the copy button next to the UUID
  5. Verify that the UUID is properly retrieved from the manager/info API response

These steps cover both frontend and backend functionality to ensure the UUID field has been properly implemented.

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • Update CHANGELOG.md
  • Commits are signed per the DCO using --signoff

- Configure manager/info endpoint to return UUID
- Ensure proper API simulation for testing

Signed-off-by: gonzaarancibia <[email protected]>
- Extract UUID from manager/info response
- Implement validation for UUID existence
- Include UUID in checkAPI endpoint response

Signed-off-by: gonzaarancibia <[email protected]>
- Implement UUID column in API table
- Add PopOver for better UUID visualization
- Include clipboard copy functionality

Signed-off-by: gonzaarancibia <[email protected]>
@gonzaarancibia gonzaarancibia linked an issue May 5, 2025 that may be closed by this pull request
1 task
@gonzaarancibia gonzaarancibia marked this pull request as ready for review May 6, 2025 16:09
@asteriscos asteriscos mentioned this pull request May 6, 2025
4 tasks
Copy link
Contributor

github-actions bot commented May 8, 2025

Wazuh Core plugin code coverage (Jest) test % values
Statements 45.3% ( 400 / 883 )
Branches 41.09% ( 157 / 382 )
Functions 43.31% ( 136 / 314 )
Lines 45.48% ( 398 / 875 )

Copy link
Contributor

github-actions bot commented May 8, 2025

Wazuh Check Updates plugin code coverage (Jest) test % values
Statements 78.72% ( 185 / 235 )
Branches 62.72% ( 69 / 110 )
Functions 61.7% ( 29 / 47 )
Lines 78.72% ( 185 / 235 )

Copy link
Contributor

github-actions bot commented May 8, 2025

Main plugin code coverage (Jest) test % values
Statements 16.89% ( 5055 / 29914 )
Branches 10.32% ( 1990 / 19280 )
Functions 17.92% ( 1300 / 7251 )
Lines 17.17% ( 4945 / 28788 )

Copy link
Contributor

@guidomodarelli guidomodarelli left a comment

Choose a reason for hiding this comment

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

Legend:
⚫ => The test hasn't started yet.
🟢 => All checks passed.
🟡 => There is at least one expected fail or skipped test and no failures.
🔴 => There is at least one failed check.
⚪ => Doesn't apply.
🔧 => Request changes.
❓ => Needs more information.

🧪 Test 2025/05/08

CR 🟢
TEST 🟢

image
image
image

Screen.Recording.2025-05-08.164853.mp4

image

Copy link
Member

@yenienserrano yenienserrano left a comment

Choose a reason for hiding this comment

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

CR: 🟢
Test: 🟡

Test

I put it in yellow because in the same table there is a similar button and they have different behavior and the function is to copy something.

Screen.Recording.2025-05-08.at.9.48.34.PM.mov

@asteriscos asteriscos merged commit 3d69e48 into 4.12.1 May 9, 2025
5 checks passed
@asteriscos asteriscos deleted the enhancement/7387-add-uuid-field-to-server-apis-table branch May 9, 2025 17:13
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.

Add UUID field to Server API's table
4 participants