[WEB-1728] Chore: Preload apis required to bootstrap the application (m…

…akeplane#5026) * chore: prefetch apis * chore: implemented cache-control * Preload links with credentials * chore: updated time in the cache and handled it based on cookie * chore: make cache private --------- Co-authored-by: gurusainath <gurusainath007@gmail.com>
smile-ttxp · Jul 8, 2024 · d8d4764 · d8d4764
1 parent fc2585b
commit d8d4764
Show file tree

Hide file tree

Showing 4 changed files with 30 additions and 0 deletions.
diff --git a/apiserver/plane/app/views/user/base.py b/apiserver/plane/app/views/user/base.py
@@ -37,6 +37,9 @@
 from plane.authentication.utils.host import user_ip
 from plane.bgtasks.user_deactivation_email_task import user_deactivation_email
 from plane.utils.host import base_host
+from django.utils.decorators import method_decorator
+from django.views.decorators.cache import cache_control
+from django.views.decorators.vary import vary_on_cookie
 
 
 class UserEndpoint(BaseViewSet):
@@ -47,6 +50,8 @@ def get_object(self):
         return self.request.user
 
     @cache_response(60 * 60)
+    @method_decorator(cache_control(private=True, max_age=12))
+    @method_decorator(vary_on_cookie)
     def retrieve(self, request):
         serialized_data = UserMeSerializer(request.user).data
         return Response(
@@ -55,6 +60,8 @@ def retrieve(self, request):
         )
 
     @cache_response(60 * 60)
+    @method_decorator(cache_control(private=True, max_age=12))
+    @method_decorator(vary_on_cookie)
     def retrieve_user_settings(self, request):
         serialized_data = UserMeSettingsSerializer(request.user).data
         return Response(serialized_data, status=status.HTTP_200_OK)
@@ -288,6 +295,8 @@ def delete(self, request, pk):
 
 
 class ProfileEndpoint(BaseAPIView):
+    @method_decorator(cache_control(private=True, max_age=12))
+    @method_decorator(vary_on_cookie)
     def get(self, request):
         profile = Profile.objects.get(user=request.user)
         serializer = ProfileSerializer(profile)

diff --git a/apiserver/plane/app/views/workspace/base.py b/apiserver/plane/app/views/workspace/base.py
@@ -44,6 +44,9 @@
     WorkspaceTheme,
 )
 from plane.utils.cache import cache_response, invalidate_cache
+from django.utils.decorators import method_decorator
+from django.views.decorators.cache import cache_control
+from django.views.decorators.vary import vary_on_cookie
 from plane.utils.constants import RESTRICTED_WORKSPACE_SLUGS
 
 
@@ -172,6 +175,8 @@ class UserWorkSpacesEndpoint(BaseAPIView):
     ]
 
     @cache_response(60 * 60 * 2)
+    @method_decorator(cache_control(private=True, max_age=12))
+    @method_decorator(vary_on_cookie)
     def get(self, request):
         fields = [
             field

diff --git a/apiserver/plane/license/api/views/instance.py b/apiserver/plane/license/api/views/instance.py
@@ -23,6 +23,8 @@
     get_configuration_value,
 )
 from plane.utils.cache import cache_response, invalidate_cache
+from django.utils.decorators import method_decorator
+from django.views.decorators.cache import cache_control
 
 
 class InstanceEndpoint(BaseAPIView):
@@ -36,6 +38,7 @@ def get_permissions(self):
         ]
 
     @cache_response(60 * 60 * 2, user=False)
+    @method_decorator(cache_control(private=True, max_age=12))
     def get(self, request):
         instance = Instance.objects.first()
 

diff --git a/web/app/layout.tsx b/web/app/layout.tsx
@@ -5,6 +5,8 @@ import "@/styles/globals.css";
 import "@/styles/command-pallette.css";
 import "@/styles/emoji.css";
 import "@/styles/react-day-picker.css";
+// helpers
+import { API_BASE_URL } from "@/helpers/common.helper";
 // local
 import { AppProvider } from "./provider";
 
@@ -36,6 +38,17 @@ export default function RootLayout({ children }: { children: React.ReactNode })
         <link rel="icon" type="image/png" sizes="16x16" href="/favicon/favicon-16x16.png" />
         <link rel="manifest" href="/site.webmanifest.json" />
         <link rel="shortcut icon" href="/favicon/favicon.ico" />
+        {/* preloading */}
+        <link rel="preload" href={`${API_BASE_URL}/api/instances/`} as="fetch" crossOrigin="use-credentials" />
+        <link rel="preload" href={`${API_BASE_URL}/api/users/me/ `} as="fetch" crossOrigin="use-credentials" />
+        <link rel="preload" href={`${API_BASE_URL}/api/users/me/profile/ `} as="fetch" crossOrigin="use-credentials" />
+        <link rel="preload" href={`${API_BASE_URL}/api/users/me/settings/ `} as="fetch" crossOrigin="use-credentials" />
+        <link
+          rel="preload"
+          href={`${API_BASE_URL}/api/users/me/workspaces/`}
+          as="fetch"
+          crossOrigin="use-credentials"
+        />
       </head>
       <body>
         <div id="context-menu-portal" />