Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cache home route #110

Merged
merged 5 commits into from
Jan 5, 2025
Merged

Conversation

GhostisLive
Copy link
Contributor

Pull Request

Description of Changes

This pull request implements file-based caching for the Flask backend to enhance the performance of the /map_route/get_all_images route. It introduces a new cache.py file and integrates caching into the Flask application.

Purpose

  • Adding a new file
  • Changing an existing file

Affected Files

List of files that are being added or changed:

  • backend/routes/cache.py (new file)
  • backend/app.py (modified to include the cache.py blueprint)
  • frontend/src/pages/user_home.jsx (minor changes for frontend logging and cache verification)

Proposed Changes

Backend Changes:

  1. New File: cache.py

    • Implements FileSystemCache for the /map_route/get_all_images route.
    • Caches the API response to a file-based storage with a configurable timeout.
  2. Changes to app.py

    • Registers the caching blueprint for seamless integration into the Flask app.

Feel free to provide feedback or suggest any changes!

Copy link

vercel bot commented Jan 4, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
walkez ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 4, 2025 3:39pm

@GhostisLive
Copy link
Contributor Author

@Rtarun3606k This should have no conflicts and unnecessary things.

@Rtarun3606k
Copy link
Owner

@Rtarun3606k This should have no conflicts and unnecessary things.

I'll review and merge tomorrow evening

Thank you 😊

@Rtarun3606k Rtarun3606k merged commit 7cb7a05 into Rtarun3606k:main Jan 5, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants