Skip to content

569 setup fastapi caching#603

Merged
LegendaryAKx3 merged 4 commits intomainfrom
569-setup-fastapi-caching
Nov 6, 2025
Merged

569 setup fastapi caching#603
LegendaryAKx3 merged 4 commits intomainfrom
569-setup-fastapi-caching

Conversation

@LegendaryAKx3
Copy link
Contributor

@LegendaryAKx3 LegendaryAKx3 commented Nov 4, 2025

Purpose

Closes #569.
Set up fastAPI caching for endpoints and create documentation for implementing caching for future endpoints.

New Changes

Explain new changes below in short bullet points.

  • Added fastapi-caching2 to requirements
  • initialize cache in lifespan.py
  • add caching to available endpoints
    Documentation for adding caching to future endpoints as they are developed will be added to the team documentation site

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.
image tests have been run for api, everything passes. The warnings are not related to my changes # 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.

@LegendaryAKx3 LegendaryAKx3 linked an issue Nov 4, 2025 that may be closed by this pull request
@github-actions
Copy link

github-actions bot commented Nov 4, 2025

Pull reviewers stats

Stats of the last 120 days for UWOrbital:

User Total reviews Time to review Total comments
Adityya-K
🥇
55
▀▀▀▀▀
1d 3h 35m
179
▀▀▀▀
camspec
🥈
26
▀▀
1d 8h
168
▀▀▀▀
kepler452b123
🥉
15
6d 6h 38m
40
Yarik-Popov
4
1d 1h 41m
40
Navtajh04
2
7d 12h 7m
▀▀
25
proprogrammer504
2
7d 1h 31m
7
sunray4
1
12d 21m
▀▀
9
c4bae
1
13d 1h 35m
▀▀▀
2

⚡️ Pull request stats

Copy link
Member

@camspec camspec left a comment

Choose a reason for hiding this comment

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

Looks good, I would modify the PR description to update it from team Notion to our docs site, and add the Purpose section. Have you tested that it caches correctly?

@camspec
Copy link
Member

camspec commented Nov 6, 2025

You can check if the endpoint was cached by looking for the response header or verifying that the endpoint was not run the second time

@LegendaryAKx3
Copy link
Contributor Author

Updated the description, caching is tested to be working (as shown in screenshot)
image

Copy link
Member

@camspec camspec left a comment

Choose a reason for hiding this comment

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

I cleaned up your description a bit, LGTM!

@LegendaryAKx3 LegendaryAKx3 merged commit a07cca9 into main Nov 6, 2025
49 checks passed
@LegendaryAKx3 LegendaryAKx3 deleted the 569-setup-fastapi-caching branch November 6, 2025 23:34
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.

Setup FastAPI caching

2 participants