Skip to content

Conversation

@hughie-coles
Copy link

Here I am applying the proxy pattern to add caching to our location API calls. This will improve performance. The caching period is also configurable

@Kilo59
Copy link
Collaborator

Kilo59 commented Aug 16, 2021

Thanks for the PR.
The caching logic on this app has long been in need of a rework!

It may take me a bit to work through the PR but I'll try to do it soon.

@hughie-coles
Copy link
Author

Honestly, I'm just doing these PRs as part of a class project, so no need to spend any effort on them

@Kilo59 Kilo59 closed this Aug 21, 2021
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