wazuh / qa-system-framework

GNU General Public License v2.0
1 stars 3 forks source link

Wazuh Handler: Create, delete assign and unassign groups to agents #60

Closed pro-akim closed 9 months ago

pro-akim commented 1 year ago

Description

Develop for qa-system-framework in WazuhHandler

Created methods

pro-akim commented 1 year ago

28/06/2023

Creation of mentioned methods were performed. After developing methods, PR was done. Some methods does not have API or Folder method.

pro-akim commented 1 year ago

Update

Checking review.

pro-akim commented 1 year ago

Update

Most of the fixes were performed. However, it will be needed to have some team communication in order to define 'validations' in each method. The status of the issue is changed to On hold

pro-akim commented 1 year ago

Update

Will be added a parameter that allows to check previous existence (agent assigned to groups or groups)

pro-akim commented 1 year ago

Update

Parameters successfully added, Tests OK Ready to review

pro-akim commented 1 year ago

Update

PR was not pushed correctly.

pro-akim commented 1 year ago

Update

Changes are done, but Linter is not working properly, the last commit was just a linting correction but the message is wrong. Ready to review

pro-akim commented 1 year ago

Update

Moved to on hold for task with higher priorities

pro-akim commented 1 year ago

Update

Moved to on hold for task with higher priorities

pro-akim commented 1 year ago

Update

Moved to on hold for task with higher priorities

pro-akim commented 1 year ago

Update

Moved to on hold for task with higher priorities

pro-akim commented 1 year ago

Fixes done.

Moved to pending review. Linter is pointing out, lines that does not show conflicts.

pro-akim commented 1 year ago

Update

Some small changes were done after linter (replacement of " for ') and parameters of 'get_agents_id' method and 'get_agent_id', documentation fixed.

pro-akim commented 9 months ago

Update

Some conflicts have been raised. Fixing them

Rebits commented 9 months ago

Delayed ETA to 22, in order to allow a review of the applied changes

pro-akim commented 9 months ago

Update

Fixes done after review.

davidjiglesias commented 9 months ago

LGTM!