Skip to content

feat(admin): remove version checker and new version alerts #120

Open
marpisco wants to merge 1 commit intomainfrom
feat/remove-version-checker
Open

feat(admin): remove version checker and new version alerts #120
marpisco wants to merge 1 commit intomainfrom
feat/remove-version-checker

Conversation

@marpisco
Copy link
Copy Markdown
Owner

@marpisco marpisco commented Apr 7, 2026

No version published will have any call-backs to home, for security reasons. This update removes version alerts, and removes the version checker by Git.

This update adds the commit below. Remember to update any manuals that have still not been ported to the platform yet to remove this reference, @marpisco.

No version published will have any call-backs to home, for security reasons. This update removes version alerts, and removes the version checker by Git.
@marpisco marpisco self-assigned this Apr 7, 2026
@marpisco marpisco added documentation Improvements or additions to documentation enhancement New feature or request labels Apr 7, 2026
Copilot AI review requested due to automatic review settings April 7, 2026 16:56
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes the admin dashboard “new version” alert feature by deleting the Git-based version check endpoint and removing the frontend code that periodically queried it, aligning with the project goal of eliminating outbound “call-backs to home” for security.

Changes:

  • Removed the admin dashboard JS/UI that displayed version/update alerts and polled every 15 minutes.
  • Deleted the admin/api/version_check.php endpoint that executed git fetch and cached results in version_cache.
  • Updated internal repository instructions to no longer reference the removed endpoint.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
admin/index.php Removes the dashboard-only version alert container and polling script.
admin/api/version_check.php Deletes the Git-backed version checking API endpoint entirely.
.github/copilot-instructions.md Removes documentation/reference to version_check.php from the admin API endpoint list.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants