Skip to content

Add local Chart.js fallback #1348

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.

Sign up for GitHub

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

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

Add local Chart.js fallback #1348

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 13 additions & 0 deletions assets/vendor/js/chart.min.js
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dashboard/README.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,8 @@ <h2>Prerequisites</h2>
<li><strong>Progress ODBC Driver</strong>: The appropriate Progress OpenEdge ODBC driver must be installed and correctly configured on the server where PHP is running. This includes setting up an ODBC Data Source Name (DSN) if you choose to connect via a system DSN, or ensuring the driver is available for DSN-less connections.</li>
<li><strong>Database Access</strong>: Network access from the web server to the Progress database server.</li>
<li><strong>Database Credentials</strong>: Valid username and password for accessing the Progress database.</li>
<li><strong>Client Internet Access</strong>: The client's web browser needs internet access to load the Chart.js library from the CDN.</li>
</ol>
<p><strong>Note</strong>: The dashboard loads the Chart.js library from a CDN, but if the CDN is unreachable, it automatically falls back to a local copy located at <code>assets/vendor/js/chart.min.js</code>. Client internet access is therefore recommended but not strictly required.</p>
<h2>Configuration</h2>
<p>The primary configuration file is <code>includes/db_connect.php</code>. You <strong>must</strong> update this file with your actual Progress database credentials and connection details.</p>
<p>Open <code>includes/db_connect.php</code> and modify the following placeholder variables:</p>
Expand Down
2 changes: 1 addition & 1 deletion dashboard/index.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
<title>Estadísticas de Visitas Web</title>
<?php require_once __DIR__ . '/../includes/head_common.php'; ?>
<link rel="stylesheet" href="../assets/css/admin_theme.css">
<script src="https://cdn.jsdelivr.net/npm/chart.js@3.7.0/dist/chart.min.js"></script> <!-- Specific version for stability -->
<script src="https://cdn.jsdelivr.net/npm/chart.js@3.7.0/dist/chart.min.js" onerror="this.onerror=null;this.src='../assets/vendor/js/chart.min.js';"></script> <!-- CDN with local fallback -->
</head>
<body class="alabaster-bg admin-page">
<?php require_once __DIR__ . '/../fragments/admin_header.php'; ?>
Expand Down
Loading