Skip to content

chore(deps): Bump tarteaucitron and remove auto-consent#3545

Open
louismaximepiton wants to merge 4 commits into
ouds/mainfrom
ouds/main-lmp-update-tarteaucitron
Open

chore(deps): Bump tarteaucitron and remove auto-consent#3545
louismaximepiton wants to merge 4 commits into
ouds/mainfrom
ouds/main-lmp-update-tarteaucitron

Conversation

@louismaximepiton
Copy link
Copy Markdown
Member

@louismaximepiton louismaximepiton commented Apr 24, 2026

Types of change

  • Non-breaking change
  • Breaking change (fix or feature that would change existing functionality and usage)

Related issues

Closes # TODO

Context & Motivation

Description

Checklists

  • I have read the contributing guidelines
  • My change follows the developer guide
  • My change pass all tests
  • My change is compatible with a responsive display
  • I have added tests (Javascript unit test or visual) to cover my changes
  • My change introduces changes to the documentation that I have updated accordingly
    • Title and DOM structure is correct
    • Links have been updated (title changes impact links)
    • CSS for the documentation
  • I have checked all states and combinations of the component with my change
  • I have checked all the impacts for the other components and core behavior (grid, reboot, utilities)

Checklist (for Core Team only)

  • The changes need to be in the migration guide
  • The changes are well displayed in Storybook (be careful if example order has changed for DSM)
  • The changes are compatible with RTL
  • Manually test browser compatibility with BrowserStack (Chrome 120, Firefox 121, Edge 120, Safari 15.6, iOS Safari, Chrome & Firefox on Android)

Progression (for Core Team only)

Live previews

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 24, 2026

Deploy Preview for boosted ready!

Name Link
🔨 Latest commit ef7ff43
🔍 Latest deploy log https://app.netlify.com/projects/boosted/deploys/6a1552137771630008bb25a8
😎 Deploy Preview https://deploy-preview-3545--boosted.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@louismaximepiton louismaximepiton marked this pull request as ready for review May 26, 2026 08:03
Copilot AI review requested due to automatic review settings May 26, 2026 08:03
@boosted-bot boosted-bot moved this from In Progress / Draft to Need Dev Review in 🟣 [Orange-Boosted-Bootstrap] PRs Board May 26, 2026
Copy link
Copy Markdown

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 updates the documentation site’s cookie-consent integration by bumping tarteaucitronjs and tightening consent behavior to remove implicit/auto-consent.

Changes:

  • Bump tarteaucitronjs from ^1.22.0 to ^1.32.0 (and update lockfile accordingly).
  • Update TarteAuCitron initialization to enable stricter privacy defaults (highPrivacy: true, alwaysNeedConsent: true).
  • Hide newly-present UI elements related to “Read more” in the TarteAuCitron UI via SCSS.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
site/src/scss/_tarteaucitron.scss Hides “Read more” UI elements in the TarteAuCitron panel.
site/src/assets/tac.js Updates TarteAuCitron init options to remove auto-consent and require consent.
package.json Bumps tarteaucitronjs version.
package-lock.json Updates resolved tarteaucitronjs package metadata to match the bump.

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

Comment thread site/src/assets/tac.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore dependencies Pull requests that update a dependency file 🗳️ feature

Projects

Status: Need Dev Review

Development

Successfully merging this pull request may close these issues.

3 participants