Skip to content

#53 limit time period for saving consens#54

Merged
webworkerJoshua merged 3 commits into
masterfrom
#53-limit-time-period-for-stored-consent
Apr 13, 2026
Merged

#53 limit time period for saving consens#54
webworkerJoshua merged 3 commits into
masterfrom
#53-limit-time-period-for-stored-consent

Conversation

@webworkerJoshua

Copy link
Copy Markdown
Owner
  • add configurable option to delete user consent after 1,3,6,12 month(s)

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds an optional retention period for client-side stored consent so that, after a configurable number of months, stored consent is discarded and the banner is shown again.

Changes:

  • Introduces consentStorageDurationMonths setting and persists a storedAt timestamp with saved consent.
  • Implements consent-expiration logic (month-based) and invalid/expired consent cleanup.
  • Exposes a new ProcessWire module config option (consent_storage_duration_months) and forwards it to the frontend settings.

Reviewed changes

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

Show a summary per file
File Description
src/js/PrivacyWire.js Adds consent storedAt timestamping, expiry calculation, and expiry enforcement when loading stored consent.
js/PrivacyWire.js Updates the built/minified modern bundle to include the new retention behavior.
js/PrivacyWire_legacy.js Updates the built/minified legacy bundle to include the new retention behavior.
PrivacyWireConfig.php Adds an admin-configurable select field for consent storage duration and a default value.
PrivacyWire.module Exposes the new config value to the frontend via PrivacyWireSettings.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/js/PrivacyWire.js
Comment thread src/js/PrivacyWire.js Outdated
Comment thread src/js/PrivacyWire.js
Comment thread src/js/PrivacyWire.js
Comment thread PrivacyWireConfig.php Outdated
@webworkerJoshua webworkerJoshua merged commit dde5f69 into master Apr 13, 2026
1 check passed
@webworkerJoshua webworkerJoshua deleted the #53-limit-time-period-for-stored-consent branch April 13, 2026 20:16
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.

limit time period for saving consens

2 participants