Skip to content

backend (Got rid of all the extra file changes)#9

Closed
WilliamRChiu wants to merge 3 commits intoUWOrbital:mainfrom
WilliamRChiu:main
Closed

backend (Got rid of all the extra file changes)#9
WilliamRChiu wants to merge 3 commits intoUWOrbital:mainfrom
WilliamRChiu:main

Conversation

@WilliamRChiu
Copy link

Purpose

Completed the GS on-boarding task. Include a screenshot of the front-end of the application.

New Changes

  • Explain new changes

Testing

  • Explain tests that you ran to verify code functionality.
  • Any functions that can be unit-tested should include a unit test in the PR. Otherwise, explain why it cannot be unit-tested.

Outstanding Changes

  • If there are non-critical changes (i.e. additional features) that can be made to this feature in the future, indicate them here.

Copy link
Contributor

@Yarik-Popov Yarik-Popov left a comment

Choose a reason for hiding this comment

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

Overall good start. Some changes are needed to improve the code

Copy link
Contributor

@Yarik-Popov Yarik-Popov left a comment

Choose a reason for hiding this comment

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

Clarification on previous requested changes

Copy link
Contributor

@Yarik-Popov Yarik-Popov left a comment

Choose a reason for hiding this comment

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

Lgtm, you are done the gs onboarding for the backend. You can pick up a backend task on the board undet the Ground Station Backend epic.

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