Skip to content

Haoyang GS Onboarding#56

Closed
HYlkdtt wants to merge 7 commits intoUWOrbital:mainfrom
HYlkdtt:main
Closed

Haoyang GS Onboarding#56
HYlkdtt wants to merge 7 commits intoUWOrbital:mainfrom
HYlkdtt:main

Conversation

@HYlkdtt
Copy link

@HYlkdtt HYlkdtt commented Oct 16, 2025

Purpose

  • I have added my name to the onboarding title (required)
  • I have added a screenshot of the logs printed by the logger middleware (required)
  • I am interested to work on the frontend (optional)

New Changes

Completed

  • Data validation for commands (see MainCommand’s validate_params_format method in backend/data/data_models.py)
  • Post endpoint for commands (see create_command function in backend/api/endpoints/command.py)
  • Delete endpoint for commands (see delete_command function in backend/api/endpoints/command.py)
  • Logger middleware (see backend/api/middlewares/logger_middleware.py)

Testing

  • Explain tests that you ran to verify code functionality.
    Through the 13tests being written with pytest already.

Copy link

@Adityya-K Adityya-K left a comment

Choose a reason for hiding this comment

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

Well done! This LGTM! Just make sure to no commit unnecessary files when pushing to the actual codebase

@Adityya-K Adityya-K closed this Oct 24, 2025
@HYlkdtt
Copy link
Author

HYlkdtt commented Oct 25, 2025

Well done! This LGTM! Just make sure to no commit unnecessary files when pushing to the actual codebase

Thanks! Sure I might have accidently run npm i and staged all files and commited :)

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.

2 participants