Skip to content

Add user consent setter/getter - #83

Merged
EmilianoSanchez merged 13 commits into
user_consent_baselinefrom
user_consent_setter_and_getter
Mar 3, 2022
Merged

Add user consent setter/getter#83
EmilianoSanchez merged 13 commits into
user_consent_baselinefrom
user_consent_setter_and_getter

Conversation

@EmilianoSanchez

@EmilianoSanchez EmilianoSanchez commented Feb 24, 2022

Copy link
Copy Markdown
Contributor

Javascript commons library

What did you accomplish?

  • Updated modular factory to support to be extended with extra properties.
  • Added set/getUserConsent methods for client-side factories.

How do we test the changes introduced in this PR?

  • Added unit tests for set/getUserConsent methods.

Extra Notes

@EmilianoSanchez
EmilianoSanchez force-pushed the user_consent_setter_and_getter branch from 3a3ff70 to 9669ad8 Compare February 24, 2022 21:37
@EmilianoSanchez EmilianoSanchez changed the title Add user consent setter/getter [WIP] Add user consent setter/getter Feb 25, 2022
@EmilianoSanchez
EmilianoSanchez merged commit 582b53a into user_consent_baseline Mar 3, 2022
@EmilianoSanchez
EmilianoSanchez deleted the user_consent_setter_and_getter branch March 3, 2022 20:02
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.

2 participants