Skip to content

Wrote all the function definitions for the manager sub system.

Kyle Joseph requested to merge 15-manager-sub-system-2 into main

Description

I added the function definitions and documentation for all the functions needed in the manager sub system. And resolved the pre-commit bugs.

Related Issue(s)

#15

Changes Made

  • Change 1: Added all the function definitions with poor documentation.
  • Change 2: Applied in-depth details for each function and resolved pre-commit bugs.
  • Change 3: Added "add member", "remove member", etc to the docstring at the top of the file.
  • Change 4: Fixed what I missed in Change 3. And added more specifics for the manager menu.

Test Plan

Same tests as any other change.

Checklist

  • Code follows the project's coding style and conventions.
  • Automated tests for the changes have been added or existing tests have been updated.
  • [] Code coverage for new or modified code is maintained or improved.
  • The code builds and runs without errors or warnings.
  • Documentation has been updated to reflect the changes made.
  • All discussions and feedback on the PR have been addressed.
  • CI/CD pipelines pass successfully, including any required checks or tests.
  • [] The PR has been reviewed and approved by the designated reviewers.

Reviewers

@reviewer1 @reviewer2

Edited by Danny Love

Merge request reports

Loading