Skip to content
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

Jump to bottom

feature: Consent Form #104

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

feature: Consent Form #104

wants to merge 2 commits into from

Conversation

jbmoelker
Copy link
Member

@jbmoelker jbmoelker commented Dec 30, 2023 •
edited
Loading

Changes

  • Adds Consent Form component with grouped consent items. Each group is enhanced with a summary checkbox, which controls and reflects the combined state of the item checkboxes.
  • Adds Switch Input component enhancing input[type="checkbox"] to an accessible switch.
  • Configurable consent groups and items
  • Consent Manager as (nano)store
  • Uses timestamp as values, so future policy changes can request new consent

WIP: https://feat-consent-form.head-start.pages.dev/consent/

Associated issue

Resolves #49.

How to test

...

Checklist

  • I have performed a self-review of my own code
  • I have made sure that my PR is easy to review (not too big, includes comments)
  • I have made updated relevant documentation files (in project README, docs/, etc)
  • I have added a decision log entry if the change affects the architecture or changes a significant technology
  • I have notified a reviewer

@jbmoelker jbmoelker changed the title feature: Consent FOrm feature: Consent Form Dec 30, 2023
@cloudflare-workers-and-pages Cloudflare Workers and Pages
Copy link

cloudflare-workers-and-pages bot commented Dec 30, 2023 •
edited
Loading

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: e03d8fa
Status: ✅  Deploy successful!
Preview URL: https://720aaf4f.head-start.pages.dev
Branch Preview URL: https://feat-consent-form.head-start.pages.dev

View logs

@jbmoelker jbmoelker self-assigned this Dec 30, 2023
jbmoelker
jbmoelker commented Jun 30, 2024
View reviewed changes
src/components/SwitchInput/SwitchInput.client.ts
@@ -0,0 +1,98 @@
/**
* Based on https://github.com/argyleink/gui-challenges/blob/main/switch/index.js
Copy link
Member Author

Choose a reason for hiding this comment

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

Is this a better alternative? https://devoorhoede.slack.com/archives/C02RU4J7H/p1709905150129279

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

@jbmoelker jbmoelker

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Consent Manager ("Cookie Banner")
1 participant
@jbmoelker