fix bug where we could decline a required app with the "decline all" btn

fixes #9, thanks.
empreinte-digitale · Apr 3, 2019 · 54ae635 · 54ae635
1 parent 00d8d1c
commit 54ae635
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/src/components/apps.js b/src/components/apps.js
@@ -29,7 +29,7 @@ export default class Apps extends React.Component {
 
         const toggle = (apps, value) => {
             apps.map((app)=>{
-                manager.updateConsent(app.name, value)
+                manager.updateConsent(app, value)
             })
         }
 

diff --git a/src/consent-manager.js b/src/consent-manager.js
@@ -65,12 +65,15 @@ export default class ConsentManager {
 
     declineAll(){
         this.config.apps.map((app) => {
-            this.updateConsent(app.name, false)
+            this.updateConsent(app, false)
         })
     }
 
-    updateConsent(name, value){
-        this.consents[name] = value
+    updateConsent(app, value){
+        if (app.required && !value) {
+            return;
+        }
+        this.consents[app.name] = value
         this.notify('consents', this.consents)
     }