Skip to content

Add analytics cookie consent banner for the docs site#336

Merged
maxnorm merged 6 commits intoPerfect-Abstractions:mainfrom
maxnorm:docs/style
Apr 4, 2026
Merged

Add analytics cookie consent banner for the docs site#336
maxnorm merged 6 commits intoPerfect-Abstractions:mainfrom
maxnorm:docs/style

Conversation

@maxnorm
Copy link
Copy Markdown
Collaborator

@maxnorm maxnorm commented Apr 4, 2026

Summary

Changes Made

Checklist

Before submitting this PR, please ensure:

  • Code follows the Solidity feature ban - No inheritance, constructors, modifiers, public/private variables, external library functions, using for directives, or selfdestruct

  • Code follows Design Principles - Readable, uses diamond storage, favors composition over inheritance

  • Code matches the codebase style - Consistent formatting, documentation, and patterns (e.g. ERC20Facet.sol)

  • Code is formatted with forge fmt

  • Existing tests pass - Run tests to be sure existing tests pass.

  • New tests are optional - If you don't provide tests for new functionality or changes then please create a new issue so this can be assigned to someone.

  • All tests pass - Run forge test and ensure everything works

  • Documentation updated - If applicable, update relevant documentation

  • Changesets — If this PR changes publishable packages (src/, cli/), changeset-bot will comment if a release note might be needed. You can add one when convenient or defer to maintainers.

Make sure to follow the contributing guidelines.

Additional Notes

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 4, 2026

Deploy Preview for compose-diamonds ready!

Name Link
🔨 Latest commit a3f643b
🔍 Latest deploy log https://app.netlify.com/projects/compose-diamonds/deploys/69d17e6fa24dfe00074bd060
😎 Deploy Preview https://deploy-preview-336--compose-diamonds.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 4, 2026

⚠️ No Changeset found

Latest commit: a3f643b

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@maxnorm maxnorm merged commit 2be9e7f into Perfect-Abstractions:main Apr 4, 2026
6 checks passed
@maxnorm maxnorm changed the title test cookie banner Add analytics cookie consent banner for the docs site Apr 4, 2026
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.

1 participant