shni/amayo
Template
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat: Pre-renderizar la navegación del dashboard para mejorar el rendimiento y evitar promesas no resueltas

Browse Source
This commit is contained in:
Shni
2025-10-14 23:54:37 -05:00
parent 66cdbfdc32
commit 4358eb20de
2 changed files with 28 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
src/server/server.ts
View File

@@ -521,6 +521,26 @@ const renderTemplate = async (
const defaultTitle = `${
locals.appName ?? pkg.name ?? "Amayo Bot"
} | Guía Completa`;
// If the caller requested the dashboard nav, render it here and pass the
// resulting HTML string to the layout to avoid printing unresolved Promises
// in the template (EJS include/await differences across environments).
let dashboardNavHtml: string | null = null;
try {
if (locals.useDashboardNav) {
const partialPath = path.join(viewsDir, "partials", "dashboard_nav.ejs");
// Render partial with same locals (async)
dashboardNavHtml = await ejs.renderFile(
partialPath,
{ ...locals },
{ async: true }
);
}
} catch (err) {
// If rendering the partial fails, log and continue — layout will handle missing nav.
console.warn("Failed rendering dashboard_nav partial:", err);
dashboardNavHtml = null;
}
const html = await ejs.renderFile(
layoutFile,
{
@@ -543,6 +563,8 @@ const renderTemplate = async (
typeof locals.useDashboardNav !== "undefined"
? locals.useDashboardNav
: false,
// Pre-rendered partial HTML (if produced above)
dashboardNav: dashboardNavHtml,
...locals,
title: locals.title ?? defaultTitle,
body: pageBody,

7
src/server/views/layouts/layout.ejs
View File

@@ -22,7 +22,12 @@
<body class="min-h-screen pixel-grid-bg pt-14" style="background-image: url('/assets/images/background.svg'); background-size: cover; background-position: center; background-attachment: fixed; background-repeat: no-repeat;">
<% if (useDashboardNav) { %>
<%- include('../partials/dashboard_nav') %>
<%# If the server pre-rendered the dashboard nav it will be available as dashboardNav (string) %>
<% if (typeof dashboardNav !== 'undefined' && dashboardNav) { %>
<%- dashboardNav %>
<% } else { %>
<%- include('../partials/dashboard_nav') %>
<% } %>
<% } else if (!hideNavbar) { %>
<%- include('../partials/navbar', { appName }) %>
<% } %>