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

Add shouldReload on ConsentManagerBuilder like optional attribute #283

Merged

Conversation

edsonjab
Copy link
Contributor

Fix issue #280

Add shouldReload on ConsnetManagerBuilder and update the documentation on the README

Copy link
Contributor

@chrisradek chrisradek left a comment

Choose a reason for hiding this comment

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

This looks great! Just had some suggestions around the doc comments.

README.md Outdated Show resolved Hide resolved
src/consent-manager-builder/index.tsx Outdated Show resolved Hide resolved
edsonjab and others added 2 commits November 9, 2022 15:43
Co-authored-by: Christopher Radek <14189820+chrisradek@users.noreply.github.com>
Co-authored-by: Christopher Radek <14189820+chrisradek@users.noreply.github.com>
@chrisradek chrisradek merged commit 8b22c38 into segmentio:master Nov 9, 2022
@edsonjab edsonjab deleted the add_shouldReload_cmBuilder branch April 1, 2023 01:58
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.

2 participants