Skip to content
This repository has been archived by the owner on Apr 3, 2024. It is now read-only.

Add new closeBehavior option to Consent Manager #48

Merged
merged 5 commits into from
Oct 9, 2019

Conversation

nettofarah
Copy link
Contributor

@nettofarah nettofarah commented Oct 9, 2019

What this PR does

  • Adds a new closeBehavior option to allow for developers to choose what's the default behavior for the x button.

Demo

demo

From the Docs:

closeBehavior

Type: enum|string

Default: dismiss

An option to determine what should be the default behavior for the x button on the consent manager banner.

Options:

  • dismiss (default) - Dismisses the banner, but don't save or change any preferences. Analytics.js won't be loaded until explicit consent is given.
  • accept - Assume consent across every category.
  • deny - Denies consent across every category.

@nettofarah nettofarah changed the base branch from master to 4.0.0 October 9, 2019 18:03
@nettofarah nettofarah merged commit cb2c4a6 into 4.0.0 Oct 9, 2019
@nettofarah nettofarah deleted the on-close-behavior branch October 9, 2019 18:03
nettofarah added a commit that referenced this pull request Oct 10, 2019
* Use hooks based API with container

* Add new `closeBehavior` option to Consent Manager (#48)

* Add option to define onCloseBehavior

* Add option to subscribe to preference changes

* Watch standalone build on dev

* Add consent option to dataset

* Document new Option

* Initial Preferences override (#49)

* Document props

* Add option to provide custom initial consent

* Change signature

* Harden type

* Swap write keys

* Add CookieView to every story

* Fix logic

* Do not generate sourceMap

* Bump size limit

* Deprecate data attributes (#51)

* Remove instructions from README
* Deprecate dataset options
* Simplify further

* Expose preferences manager (#52)

* Add changelog (#53)

* Update version;
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant