fix remove provider from backup and #7167

packages/taler-wallet-webextension/src/compat.ts

@@ -81,6 +81,7 @@ export function getPermissionsApi(): CrossBrowserPermissionsApi {
   ) {
     return {
       addPermissionsListener: () => {
+        console.log("not supported for firefox")
         // Not supported yet.
       },
       contains: myBrowser.permissions.contains,

packages/taler-wallet-webextension/src/components/TransactionItem.tsx

@@ -111,7 +111,7 @@ export function TransactionItem(props: { tx: Transaction }): VNode {
           debitCreditIndicator={"debit"}
           title={tx.targetPaytoUri}
           timestamp={tx.timestamp}
-          iconPath={imageRefresh}
+          iconPath={imageBank}
           pending={tx.pending}
         />
       );

packages/taler-wallet-webextension/src/hooks/useExtendedPermissions.ts

@@ -19,11 +19,11 @@ import * as wxApi from "../wxApi";
 import { getPermissionsApi } from "../compat";
 import { getReadRequestPermissions } from "../permissions";
 
-export function useExtendedPermissions(): [boolean, () => void] {
+export function useExtendedPermissions(): [boolean, () => Promise<void>] {
   const [enabled, setEnabled] = useState(false);
 
-  const toggle = () => {
+  const toggle = async () => {
-    handleExtendedPerm(enabled, setEnabled)
+    return handleExtendedPerm(enabled, setEnabled)
   };
 
   useEffect(() => {
@@ -36,22 +36,29 @@ export function useExtendedPermissions(): [boolean, () => void] {
   return [enabled, toggle];
 }
 
-function handleExtendedPerm(isEnabled: boolean, onChange: (value: boolean) => void): void {
+async function handleExtendedPerm(isEnabled: boolean, onChange: (value: boolean) => void): Promise<void> {
   if (!isEnabled) {
     // We set permissions here, since apparently FF wants this to be done
     // as the result of an input event ...
-    getPermissionsApi().request(getReadRequestPermissions(), async (granted: boolean) => {
+    return new Promise<void>((res) => {
-      if (chrome.runtime.lastError) {
+      getPermissionsApi().request(getReadRequestPermissions(), async (granted: boolean) => {
-        console.error("error requesting permissions");
+        console.log("permissions granted:", granted);
-        console.error(chrome.runtime.lastError);
+        if (chrome.runtime.lastError) {
-        onChange(false);
+          console.error("error requesting permissions");
-        return;
+          console.error(chrome.runtime.lastError);
-      }
+          onChange(false);
-      console.log("permissions granted:", granted);
+          return;
-      const res = await wxApi.setExtendedPermissions(granted);
+        }
-      onChange(res.newValue);
+        try {
-    });
+          const res = await wxApi.setExtendedPermissions(granted);
-  } else {
+          onChange(res.newValue);
-    wxApi.setExtendedPermissions(false).then(r => onChange(r.newValue));
+        } finally {
+          res()
+        }
+
+      });
+    })
   }
+  await wxApi.setExtendedPermissions(false).then(r => onChange(r.newValue));
+  return
 }

packages/taler-wallet-webextension/src/wallet/ProviderDetailPage.tsx

@@ -83,7 +83,7 @@ export function ProviderDetailPage({ pid: providerURL, onBack }: Props): VNode {
     <ProviderView
       info={state.response}
       onSync={async () => wxApi.syncOneProvider(providerURL)}
-      onDelete={async () => wxApi.syncOneProvider(providerURL).then(onBack)}
+      onDelete={async () => wxApi.removeProvider(providerURL).then(onBack)}
       onBack={onBack}
       onExtend={() => {
         null;