Skip to content

Conversation

@Scr4tch587
Copy link
Contributor

Purpose

Closes #344 .
Explain the purpose of the PR here if it doesn't match the linked issue. Be sure to add a comment in the linked issue explaining the changes.

New Changes

  • Installed react query
  • Switched all useEffect to useQuery or useMutation
  • Added test endpoints to the backend that return hardcoded data for MCC testing

Testing

Explain tests that you ran to verify code functionality.

  • I have unit-tested this PR. Otherwise, explain why it cannot be unit-tested.
  • I have tested this PR on a board if the code will run on a board (Only required for firmware developers).
  • I have tested this PR by running the ARO website (Only required if the code will impact the ARO website).
  • I have tested this PR by running the MCC website (Only required if the code will impact the MCC website).
  • I have included screenshots of the tests performed below.

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.

image image image

Copy link
Contributor

@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.

Minor changes, will review once more once confirmation is received on if we want to merge in the mocks or not.

load_dotenv()


print(environ.get("GS_DATABASE_LOCATION"))
Copy link
Contributor

Choose a reason for hiding this comment

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

I think you should remove these print statements since they seem to be for debugging

@github-actions
Copy link

github-actions bot commented Oct 1, 2025

Pull reviewers stats

Stats of the last 120 days for UWOrbital:

User Total reviews Time to review Total comments
Adityya-K 38 20h 14m 116
kepler452b123 18 4d 7h 38m 70
camspec 12 10h 1m 78
Yarik-Popov 7 1d 1h 41m 58
Navtajh04 6 5d 1h 23m 68

⚡️ Pull request stats

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.

Switch to using react-query instead of raw fetch

2 participants