add a comment about the notifications

we had a bug related to this call, so adding a note to future development to make sure we're aware of the ramifications that this can cause.
e-mission · Oct 18, 2023 · 0584278 · 0584278
1 parent 416a111
commit 0584278
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/www/js/splash/startprefs.ts b/www/js/splash/startprefs.ts
@@ -23,6 +23,9 @@ let _req_consent;
 let _curr_consented;
 
 function writeConsentToNative() {
+  //note that this calls to the notification API, 
+  //so should not be called until we have notification permissions
+  //see https://github.com/e-mission/e-mission-docs/issues/1006
   return window['cordiva'].plugins.BEMDataCollection.markConsented(_req_consent);
 };