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

Added caching to local UI pages #267

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rutujaac
Copy link
Collaborator

@rutujaac rutujaac commented Mar 13, 2024

Added caching for pebblo-ui directory except pebblo-ui/static directory
Added middleware to redirect non trailing '/' requests to trailing '/' routes.

@rutujaac rutujaac marked this pull request as draft March 13, 2024 11:25
@shreyas-damle
Copy link
Collaborator

@rutujaac When I tried locally with this branch, Local UI is working fine, but APIs are giving errors when app is executed.

INFO:     ::1:63714 - "POST /v1/app/discover HTTP/1.1" 307 Temporary Redirect
INFO:     ::1:63714 - "POST /v1/app/discover/ HTTP/1.1" 405 Method Not Allowed
INFO:     ::1:63715 - "POST /v1/loader/doc HTTP/1.1" 307 Temporary Redirect
INFO:     ::1:63715 - "POST /v1/loader/doc/ HTTP/1.1" 405 Method Not Allowed

We need to solve this problem before going ahead with the solution implemented in this PR.

@Raj725 Raj725 removed their request for review September 13, 2024 04:28
Copy link

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