cozy-harvest-lib 8.4.2 → 9.1.0

package/CHANGELOG.md

@@ -3,6 +3,49 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [9.1.0](https://github.com/cozy/cozy-libs/compare/cozy-harvest-lib@9.0.0...cozy-harvest-lib@9.1.0) (2022-05-02)
+
+
+### Bug Fixes
+
+* Multiple fixes after review ([0351500](https://github.com/cozy/cozy-libs/commit/0351500e1845df6ef4a31211aec6061090ccf66e))
+
+
+### Features
+
+* Open InAppBrowser via cozy-intent when in flaghip app ([5ee31ea](https://github.com/cozy/cozy-libs/commit/5ee31eafcb62e23a6242386bb6b09312f778e871))
+
+
+
+
+
+# [9.0.0](https://github.com/cozy/cozy-libs/compare/cozy-harvest-lib@8.4.3...cozy-harvest-lib@9.0.0) (2022-04-29)
+
+
+### Bug Fixes
+
+* Check if we need to display the unlock form ([fe230ce](https://github.com/cozy/cozy-libs/commit/fe230ce0791c511c31098be47e633408bceb9c30))
+* Upgrade cozy-keys-lib requirement ([c4eb57f](https://github.com/cozy/cozy-libs/commit/c4eb57f22b4a86d42c4422619a158605d894edcf))
+* VaultClient can be undefined at first ([8a6f02b](https://github.com/cozy/cozy-libs/commit/8a6f02b9e9c161622afedff88ba4805468ee6f59))
+
+
+### BREAKING CHANGES
+
+* cozy-keys-lib >= 4.1.9 is now
+required.
+
+
+
+
+
+## [8.4.3](https://github.com/cozy/cozy-libs/compare/cozy-harvest-lib@8.4.2...cozy-harvest-lib@8.4.3) (2022-04-28)
+
+**Note:** Version bump only for package cozy-harvest-lib
+
+
+
+
+
 ## [8.4.2](https://github.com/cozy/cozy-libs/compare/cozy-harvest-lib@8.4.1...cozy-harvest-lib@8.4.2) (2022-04-26)
 
 

package/dist/components/InAppBrowser.js

@@ -0,0 +1,65 @@
+import _asyncToGenerator from "@babel/runtime/helpers/asyncToGenerator";
+import _regeneratorRuntime from "@babel/runtime/regenerator";
+import { useEffect } from 'react';
+import PropTypes from 'prop-types';
+import { useWebviewIntent } from 'cozy-intent';
+import logger from '../logger';
+
+var InAppBrowser = function InAppBrowser(_ref) {
+  var url = _ref.url,
+      onClose = _ref.onClose;
+  var webviewIntent = useWebviewIntent();
+  useEffect(function () {
+    function insideEffect() {
+      return _insideEffect.apply(this, arguments);
+    }
+
+    function _insideEffect() {
+      _insideEffect = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee() {
+        var result;
+        return _regeneratorRuntime.wrap(function _callee$(_context) {
+          while (1) {
+            switch (_context.prev = _context.next) {
+              case 0:
+                if (!webviewIntent) {
+                  _context.next = 5;
+                  break;
+                }
+
+                _context.next = 3;
+                return webviewIntent.call('showInAppBrowser', {
+                  url: url
+                });
+
+              case 3:
+                result = _context.sent;
+
+                if ((result === null || result === void 0 ? void 0 : result.type) === 'cancel' && onClose) {
+                  onClose();
+                } else if ((result === null || result === void 0 ? void 0 : result.type) !== 'dismiss') {
+                  logger.error('Unexpected InAppBrowser result', result);
+                }
+
+              case 5:
+              case "end":
+                return _context.stop();
+            }
+          }
+        }, _callee);
+      }));
+      return _insideEffect.apply(this, arguments);
+    }
+
+    insideEffect();
+    return function cleanup() {
+      webviewIntent.call('closeInAppBrowser');
+    };
+  }, [webviewIntent, url, onClose]);
+  return null;
+};
+
+InAppBrowser.propTypes = {
+  url: PropTypes.string.isRequired,
+  onClose: PropTypes.func
+};
+export default InAppBrowser;

package/dist/components/InAppBrowser.spec.js

@@ -0,0 +1,104 @@
+import _asyncToGenerator from "@babel/runtime/helpers/asyncToGenerator";
+import _regeneratorRuntime from "@babel/runtime/regenerator";
+import React from 'react';
+import { render, waitFor } from '@testing-library/react';
+import InAppBrowser from './InAppBrowser';
+import { WebviewIntentProvider } from 'cozy-intent';
+describe('InAppBrowser', function () {
+  it('should call showInAppBrowser', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee() {
+    var url, intentCall, webviewService;
+    return _regeneratorRuntime.wrap(function _callee$(_context) {
+      while (1) {
+        switch (_context.prev = _context.next) {
+          case 0:
+            url = 'https://test.url';
+            intentCall = jest.fn();
+            webviewService = {
+              call: intentCall
+            };
+            intentCall.mockResolvedValue({
+              type: 'dismiss'
+            });
+            render( /*#__PURE__*/React.createElement(WebviewIntentProvider, {
+              webviewService: webviewService
+            }, /*#__PURE__*/React.createElement(InAppBrowser, {
+              url: url
+            })));
+            expect(webviewService.call).toHaveBeenNthCalledWith(1, 'showInAppBrowser', {
+              url: url
+            });
+
+          case 6:
+          case "end":
+            return _context.stop();
+        }
+      }
+    }, _callee);
+  })));
+  it('should call onClose when user closes the inAppBrowser in app', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee2() {
+    var url, intentCall, webviewService, onClose;
+    return _regeneratorRuntime.wrap(function _callee2$(_context2) {
+      while (1) {
+        switch (_context2.prev = _context2.next) {
+          case 0:
+            url = 'https://test.url';
+            intentCall = jest.fn();
+            webviewService = {
+              call: intentCall
+            };
+            onClose = jest.fn();
+            intentCall.mockResolvedValue({
+              type: 'cancel'
+            });
+            render( /*#__PURE__*/React.createElement(WebviewIntentProvider, {
+              webviewService: webviewService
+            }, /*#__PURE__*/React.createElement(InAppBrowser, {
+              url: url,
+              onClose: onClose
+            })));
+            _context2.next = 8;
+            return waitFor(function () {
+              return expect(onClose).toHaveBeenCalledTimes(1);
+            });
+
+          case 8:
+            expect(onClose).toHaveBeenCalledWith();
+
+          case 9:
+          case "end":
+            return _context2.stop();
+        }
+      }
+    }, _callee2);
+  })));
+  it('should call closeInAppBrowser when the component is unmounted', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee3() {
+    var url, intentCall, webviewService, _render, unmount;
+
+    return _regeneratorRuntime.wrap(function _callee3$(_context3) {
+      while (1) {
+        switch (_context3.prev = _context3.next) {
+          case 0:
+            url = 'https://test.url';
+            intentCall = jest.fn();
+            webviewService = {
+              call: intentCall
+            };
+            intentCall.mockResolvedValue({
+              type: 'dismiss'
+            });
+            _render = render( /*#__PURE__*/React.createElement(WebviewIntentProvider, {
+              webviewService: webviewService
+            }, /*#__PURE__*/React.createElement(InAppBrowser, {
+              url: url
+            }))), unmount = _render.unmount;
+            unmount();
+            expect(webviewService.call).toHaveBeenNthCalledWith(2, 'closeInAppBrowser');
+
+          case 7:
+          case "end":
+            return _context3.stop();
+        }
+      }
+    }, _callee3);
+  })));
+});

package/dist/components/KonnectorConfiguration/ConfigurationTab/index.js

@@ -6,7 +6,7 @@ import PropTypes from 'prop-types';
 import cx from 'classnames';
 import { useClient } from 'cozy-client';
 import { Account } from 'cozy-doctypes';
-import { useVaultClient } from 'cozy-keys-lib';
+import { useVaultClient, CozyUtils } from 'cozy-keys-lib';
 import Button from 'cozy-ui/transpiled/react/Button';
 import Spinner from 'cozy-ui/transpiled/react/Spinner';
 import palette from 'cozy-ui/transpiled/react/palette';
@@ -118,7 +118,10 @@ var ConfigurationTab = function ConfigurationTab(_ref2) {
 
               showUnlockForm({
                 closable: true,
-                onUnlock: handleUnlockForDeletion
+                onUnlock: handleUnlockForDeletion,
+                addCheckShouldUnlock: function addCheckShouldUnlock() {
+                  return CozyUtils.checkHasInstalledExtension(client);
+                }
               });
               _context.next = 8;
               break;

package/dist/components/OAuthWindow.js

@@ -15,9 +15,11 @@ import PropTypes from 'prop-types';
 import { withClient } from 'cozy-client';
 import { translate } from 'cozy-ui/transpiled/react/I18n';
 import CozyRealtime from 'cozy-realtime';
+import { isFlagshipApp } from 'cozy-device-helper';
 import { prepareOAuth, checkOAuthData, terminateOAuth, OAUTH_REALTIME_CHANNEL } from '../helpers/oauth'; // TODO use PopUp from cozy-ui
 
 import Popup from './Popup';
+import InAppBrowser from './InAppBrowser';
 var OAUTH_POPUP_HEIGHT = 800;
 var OAUTH_POPUP_WIDTH = 800;
 /**
@@ -177,14 +179,17 @@ export var OAuthWindow = /*#__PURE__*/function (_PureComponent) {
       var _this$state = this.state,
           oAuthUrl = _this$state.oAuthUrl,
           succeed = _this$state.succeed;
-      return oAuthUrl && !succeed && /*#__PURE__*/React.createElement(Popup, {
+      return oAuthUrl && !succeed && (!isFlagshipApp() ? /*#__PURE__*/React.createElement(Popup, {
         url: oAuthUrl,
         height: OAUTH_POPUP_HEIGHT,
         width: OAUTH_POPUP_WIDTH,
         onClose: this.handleClose,
         onUrlChange: this.handleUrlChange,
         title: t("oauth.window.title")
-      });
+      }) : /*#__PURE__*/React.createElement(InAppBrowser, {
+        url: oAuthUrl,
+        onClose: this.handleClose
+      }));
     }
   }]);
 

package/dist/components/TriggerManager.js

@@ -1,5 +1,6 @@
 import _extends from "@babel/runtime/helpers/extends";
 import _objectWithoutProperties from "@babel/runtime/helpers/objectWithoutProperties";
+import _defineProperty from "@babel/runtime/helpers/defineProperty";
 import _asyncToGenerator from "@babel/runtime/helpers/asyncToGenerator";
 import _classCallCheck from "@babel/runtime/helpers/classCallCheck";
 import _createClass from "@babel/runtime/helpers/createClass";
@@ -9,6 +10,9 @@ import _possibleConstructorReturn from "@babel/runtime/helpers/possibleConstruct
 import _getPrototypeOf from "@babel/runtime/helpers/getPrototypeOf";
 var _excluded = ["onLaunch", "onSuccess", "onLoginSuccess", "onError", "initialTrigger"],
     _excluded2 = ["vaultUnlockFormProps"];
+
+var _DumbTriggerManager$p;
+
 import _regeneratorRuntime from "@babel/runtime/regenerator";
 
 function _createSuper(Derived) { var hasNativeReflectConstruct = _isNativeReflectConstruct(); return function _createSuperInternal() { var Super = _getPrototypeOf(Derived), result; if (hasNativeReflectConstruct) { var NewTarget = _getPrototypeOf(this).constructor; result = Reflect.construct(Super, arguments, NewTarget); } else { result = Super.apply(this, arguments); } return _possibleConstructorReturn(this, result); }; }
@@ -24,7 +28,7 @@ import { Account } from 'cozy-doctypes';
 import { translate } from 'cozy-ui/transpiled/react/I18n';
 import Spinner from 'cozy-ui/transpiled/react/Spinner';
 import { ModalBackButton } from 'cozy-ui/transpiled/react/Modal';
-import { CipherType } from 'cozy-keys-lib';
+import { CipherType, withVaultUnlockContext, VaultUnlockPlaceholder, VaultUnlockProvider, useVaultClient, CozyUtils } from 'cozy-keys-lib';
 import AccountForm from './AccountForm';
 import OAuthForm from './OAuthForm';
 import { fetchAccount } from '../connections/accounts';
@@ -34,7 +38,6 @@ import manifest from '../helpers/manifest';
 import logger from '../logger';
 import { findKonnectorPolicy } from '../konnector-policies';
 import withConnectionFlow from '../models/withConnectionFlow';
-import { withVaultUnlockContext, VaultUnlockPlaceholder, VaultUnlockProvider } from 'cozy-keys-lib';
 import HarvestVaultProvider from './HarvestVaultProvider';
 var IDLE = 'IDLE';
 var RUNNING = 'RUNNING';
@@ -301,13 +304,13 @@ export var DumbTriggerManager = /*#__PURE__*/function (_Component) {
     key: "componentDidMount",
     value: function () {
       var _componentDidMount = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee3() {
-        var _this$props3, konnector, showUnlockForm, onVaultDismiss, vaultClosable, vaultClient, konnectorPolicy, isVaultLocked;
+        var _this$props3, konnector, showUnlockForm, onVaultDismiss, vaultClosable, vaultClient, client, konnectorPolicy, isVaultLocked;
 
         return _regeneratorRuntime.wrap(function _callee3$(_context3) {
           while (1) {
             switch (_context3.prev = _context3.next) {
               case 0:
-                _this$props3 = this.props, konnector = _this$props3.konnector, showUnlockForm = _this$props3.showUnlockForm, onVaultDismiss = _this$props3.onVaultDismiss, vaultClosable = _this$props3.vaultClosable, vaultClient = _this$props3.vaultClient;
+                _this$props3 = this.props, konnector = _this$props3.konnector, showUnlockForm = _this$props3.showUnlockForm, onVaultDismiss = _this$props3.onVaultDismiss, vaultClosable = _this$props3.vaultClosable, vaultClient = _this$props3.vaultClient, client = _this$props3.client;
                 konnectorPolicy = findKonnectorPolicy(konnector);
 
                 if (!konnectorPolicy.saveInVault) {
@@ -333,7 +336,10 @@ export var DumbTriggerManager = /*#__PURE__*/function (_Component) {
                   showUnlockForm({
                     onDismiss: onVaultDismiss,
                     closable: vaultClosable,
-                    onUnlock: this.handleVaultUnlock
+                    onUnlock: this.handleVaultUnlock,
+                    addCheckShouldUnlock: function addCheckShouldUnlock() {
+                      return CozyUtils.checkHasInstalledExtension(client);
+                    }
                   });
                 } else {
                   this.handleVaultUnlock();
@@ -561,7 +567,7 @@ export var DumbTriggerManager = /*#__PURE__*/function (_Component) {
 
   return DumbTriggerManager;
 }(Component);
-DumbTriggerManager.propTypes = {
+DumbTriggerManager.propTypes = (_DumbTriggerManager$p = {
   /**
    * Account document. Used to get initial form values.
    * If no account is passed, AccountForm will use empty initial values.
@@ -594,11 +600,29 @@ DumbTriggerManager.propTypes = {
    * Whether the vault will be closable or not.
    * @type {Boolean}
    */
-  vaultClosable: PropTypes.bool
-};
+  vaultClosable: PropTypes.bool,
+
+  /**
+   *
+   */
+  vaultClient: PropTypes.object,
+  client: PropTypes.object,
+  onError: PropTypes.func,
+  showUnlockForm: PropTypes.func
+}, _defineProperty(_DumbTriggerManager$p, "onVaultDismiss", PropTypes.func), _defineProperty(_DumbTriggerManager$p, "error", PropTypes.any), _defineProperty(_DumbTriggerManager$p, "showError", PropTypes.bool), _defineProperty(_DumbTriggerManager$p, "fieldOptions", PropTypes.object), _defineProperty(_DumbTriggerManager$p, "flow", PropTypes.object), _defineProperty(_DumbTriggerManager$p, "flowState", PropTypes.object), _DumbTriggerManager$p);
 var TriggerManager = compose(translate(), withClient, withVaultUnlockContext, withConnectionFlow())(DumbTriggerManager); // TriggerManager is exported wrapped in FlowProvider to avoid breaking changes.
 
 var LegacyTriggerManager = function LegacyTriggerManager(props) {
+  // Since the 4.1.0 of cozy-keys-lib, we
+  // render children even if vaultClient is
+  // not defined yet. In that case we we were
+  // displaying TriggerManager without vaultClient.
+  // It was raising an error.
+  // The current fix, is to not display the
+  // TriggerManager when vaultClient is null.
+  var vaultClient = useVaultClient();
+  if (!vaultClient) return null;
+
   var onLaunch = props.onLaunch,
       onSuccess = props.onSuccess,
       onLoginSuccess = props.onLoginSuccess,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cozy-harvest-lib",
-  "version": "8.4.2",
+  "version": "9.1.0",
   "description": "Provides logic, modules and components for Cozy's harvest applications.",
   "main": "dist/index.js",
   "author": "Cozy",
@@ -53,10 +53,11 @@
     "babel-plugin-inline-react-svg": "1.1.2",
     "babel-preset-cozy-app": "^2.0.2",
     "cozy-client": "27.17.0",
-    "cozy-device-helper": "^1.18.0",
+    "cozy-device-helper": "^2.0.0",
     "cozy-flags": "^2.8.7",
-    "cozy-keys-lib": "3.8.0",
-    "cozy-realtime": "^4.0.7",
+    "cozy-intent": "^1.17.1",
+    "cozy-keys-lib": "^4.1.9",
+    "cozy-realtime": "^4.0.8",
     "cozy-ui": "60.6.0",
     "enzyme": "3.11.0",
     "enzyme-adapter-react-16": "1.15.6",
@@ -78,12 +79,13 @@
     "cozy-client": ">=27.17.0",
     "cozy-device-helper": ">=1.10.2",
     "cozy-flags": ">=2.3.5",
-    "cozy-keys-lib": ">=3.7.0",
+    "cozy-intent": ">=1.14.1",
+    "cozy-keys-lib": ">=4.1.9",
     "cozy-realtime": ">=3.12.2",
     "cozy-ui": ">=60.6.0",
     "leaflet": "^1.7.1",
     "react-router-dom": "^5.0.1"
   },
   "sideEffects": false,
-  "gitHead": "b60562af8d20b51df3d7b0d35bb3e9a3d25d318f"
+  "gitHead": "2e308c267a13adc3cd826639e3058fe35770d830"
 }

package/src/components/AccountForm/index.jsx

@@ -132,7 +132,9 @@ export class AccountForm extends PureComponent {
   handleSubmit(values, form) {
     const { account, konnector } = this.props
 
-    const identifier = manifest.getIdentifier(manifest.sanitizeFields(konnector.fields))
+    const identifier = manifest.getIdentifier(
+      manifest.sanitizeFields(konnector.fields)
+    )
     if (
       account &&
       account.auth[identifier] &&

package/src/components/InAppBrowser.jsx

@@ -0,0 +1,33 @@
+import { useEffect } from 'react'
+import PropTypes from 'prop-types'
+import { useWebviewIntent } from 'cozy-intent'
+import logger from '../logger'
+
+const InAppBrowser = ({ url, onClose }) => {
+  const webviewIntent = useWebviewIntent()
+
+  useEffect(() => {
+    async function insideEffect() {
+      if (webviewIntent) {
+        const result = await webviewIntent.call('showInAppBrowser', { url })
+        if (result?.type === 'cancel' && onClose) {
+          onClose()
+        } else if (result?.type !== 'dismiss') {
+          logger.error('Unexpected InAppBrowser result', result)
+        }
+      }
+    }
+    insideEffect()
+    return function cleanup() {
+      webviewIntent.call('closeInAppBrowser')
+    }
+  }, [webviewIntent, url, onClose])
+  return null
+}
+
+InAppBrowser.propTypes = {
+  url: PropTypes.string.isRequired,
+  onClose: PropTypes.func
+}
+
+export default InAppBrowser

package/src/components/InAppBrowser.spec.jsx

@@ -0,0 +1,60 @@
+import React from 'react'
+import { render, waitFor } from '@testing-library/react'
+import InAppBrowser from './InAppBrowser'
+import { WebviewIntentProvider } from 'cozy-intent'
+
+describe('InAppBrowser', () => {
+  it('should call showInAppBrowser', async () => {
+    const url = 'https://test.url'
+    const intentCall = jest.fn()
+    const webviewService = {
+      call: intentCall
+    }
+    intentCall.mockResolvedValue({type: 'dismiss'})
+    render(
+      <WebviewIntentProvider webviewService={webviewService}>
+        <InAppBrowser url={url} />
+      </WebviewIntentProvider>
+    )
+
+    expect(webviewService.call).toHaveBeenNthCalledWith(1, 'showInAppBrowser', {
+      url
+    })
+  })
+  it('should call onClose when user closes the inAppBrowser in app', async () => {
+    const url = 'https://test.url'
+    const intentCall = jest.fn()
+    const webviewService = {
+      call: intentCall
+    }
+    const onClose = jest.fn()
+
+    intentCall.mockResolvedValue({type: 'cancel'})
+    render(
+      <WebviewIntentProvider webviewService={webviewService}>
+        <InAppBrowser url={url} onClose={onClose} />
+      </WebviewIntentProvider>
+    )
+
+    await waitFor(() => expect(onClose).toHaveBeenCalledTimes(1))
+    expect(onClose).toHaveBeenCalledWith()
+
+
+  })
+  it('should call closeInAppBrowser when the component is unmounted', async () => {
+    const url = 'https://test.url'
+    const intentCall = jest.fn()
+    const webviewService = {
+      call: intentCall
+    }
+    intentCall.mockResolvedValue({type: 'dismiss'})
+    const { unmount } = render(
+      <WebviewIntentProvider webviewService={webviewService}>
+        <InAppBrowser url={url} />
+      </WebviewIntentProvider>
+    )
+
+    unmount()
+    expect(webviewService.call).toHaveBeenNthCalledWith(2, 'closeInAppBrowser')
+  })
+})

package/src/components/KonnectorConfiguration/ConfigurationTab/index.jsx

@@ -4,7 +4,7 @@ import cx from 'classnames'
 
 import { useClient } from 'cozy-client'
 import { Account } from 'cozy-doctypes'
-import { useVaultClient } from 'cozy-keys-lib'
+import { useVaultClient, CozyUtils } from 'cozy-keys-lib'
 
 import Button from 'cozy-ui/transpiled/react/Button'
 import Spinner from 'cozy-ui/transpiled/react/Spinner'
@@ -96,7 +96,8 @@ const ConfigurationTab = ({
     if (konnectorPolicy.saveInVault) {
       showUnlockForm({
         closable: true,
-        onUnlock: handleUnlockForDeletion
+        onUnlock: handleUnlockForDeletion,
+        addCheckShouldUnlock: () => CozyUtils.checkHasInstalledExtension(client)
       })
     } else {
       await handleDeleteAccount()

package/src/components/NewAccountModal.spec.jsx

@@ -58,7 +58,9 @@ describe('NewAccountModal', () => {
       </AppLike>
     )
     onLoginSuccessFn(konnectorTrigger)
-    expect(replaceHistory).toHaveBeenCalledWith('/accounts/accountNumber/success')
+    expect(replaceHistory).toHaveBeenCalledWith(
+      '/accounts/accountNumber/success'
+    )
   })
   it('should redirect to route without success on login success for client triggers', () => {
     render(
@@ -85,7 +87,9 @@ describe('NewAccountModal', () => {
       </AppLike>
     )
     onSuccessFn(konnectorTrigger)
-    expect(replaceHistory).toHaveBeenCalledWith('/accounts/accountNumber/success')
+    expect(replaceHistory).toHaveBeenCalledWith(
+      '/accounts/accountNumber/success'
+    )
   })
   it('should redirect to route without success on success for client triggers', () => {
     render(

package/src/components/OAuthWindow.jsx

@@ -4,6 +4,7 @@ import PropTypes from 'prop-types'
 import { withClient } from 'cozy-client'
 import { translate } from 'cozy-ui/transpiled/react/I18n'
 import CozyRealtime from 'cozy-realtime'
+import { isFlagshipApp } from 'cozy-device-helper'
 
 import {
   prepareOAuth,
@@ -13,6 +14,7 @@ import {
 } from '../helpers/oauth'
 // TODO use PopUp from cozy-ui
 import Popup from './Popup'
+import InAppBrowser from './InAppBrowser'
 
 const OAUTH_POPUP_HEIGHT = 800
 const OAUTH_POPUP_WIDTH = 800
@@ -141,7 +143,8 @@ export class OAuthWindow extends PureComponent {
     const { oAuthUrl, succeed } = this.state
     return (
       oAuthUrl &&
-      !succeed && (
+      !succeed &&
+      (!isFlagshipApp() ? (
         <Popup
           url={oAuthUrl}
           height={OAUTH_POPUP_HEIGHT}
@@ -150,7 +153,9 @@ export class OAuthWindow extends PureComponent {
           onUrlChange={this.handleUrlChange}
           title={t(`oauth.window.title`)}
         />
-      )
+      ) : (
+        <InAppBrowser url={oAuthUrl} onClose={this.handleClose} />
+      ))
     )
   }
 }

package/src/components/TriggerManager.jsx

@@ -10,7 +10,14 @@ import { translate } from 'cozy-ui/transpiled/react/I18n'
 import Spinner from 'cozy-ui/transpiled/react/Spinner'
 import { ModalBackButton } from 'cozy-ui/transpiled/react/Modal'
 
-import { CipherType } from 'cozy-keys-lib'
+import {
+  CipherType,
+  withVaultUnlockContext,
+  VaultUnlockPlaceholder,
+  VaultUnlockProvider,
+  useVaultClient,
+  CozyUtils
+} from 'cozy-keys-lib'
 
 import AccountForm from './AccountForm'
 import OAuthForm from './OAuthForm'
@@ -21,11 +28,6 @@ import manifest from '../helpers/manifest'
 import logger from '../logger'
 import { findKonnectorPolicy } from '../konnector-policies'
 import withConnectionFlow from '../models/withConnectionFlow'
-import {
-  withVaultUnlockContext,
-  VaultUnlockPlaceholder,
-  VaultUnlockProvider
-} from 'cozy-keys-lib'
 import HarvestVaultProvider from './HarvestVaultProvider'
 
 const IDLE = 'IDLE'
@@ -210,7 +212,8 @@ export class DumbTriggerManager extends Component {
       showUnlockForm,
       onVaultDismiss,
       vaultClosable,
-      vaultClient
+      vaultClient,
+      client
     } = this.props
     const konnectorPolicy = findKonnectorPolicy(konnector)
     if (konnectorPolicy.saveInVault) {
@@ -224,7 +227,9 @@ export class DumbTriggerManager extends Component {
         showUnlockForm({
           onDismiss: onVaultDismiss,
           closable: vaultClosable,
-          onUnlock: this.handleVaultUnlock
+          onUnlock: this.handleVaultUnlock,
+          addCheckShouldUnlock: () =>
+            CozyUtils.checkHasInstalledExtension(client)
         })
       } else {
         this.handleVaultUnlock()
@@ -410,7 +415,29 @@ DumbTriggerManager.propTypes = {
    * Whether the vault will be closable or not.
    * @type {Boolean}
    */
-  vaultClosable: PropTypes.bool
+  vaultClosable: PropTypes.bool,
+  /**
+   *
+   */
+  vaultClient: PropTypes.object,
+  client: PropTypes.object,
+  onError: PropTypes.func,
+  showUnlockForm: PropTypes.func,
+  onVaultDismiss: PropTypes.func,
+  error: PropTypes.any,
+   /**
+   * Indicates if the AccountForm has to show errors. Sometimes errors may be
+   * displayed elsewhere. However, a KonnectorJobError corresponding to a login
+   * error is always displayed.
+   * @type {Boolean}
+   */
+    showError: PropTypes.bool,
+  /**
+   * Used to have options on fields (forceEncryptedPlaceholder or focus)
+   */
+  fieldOptions: PropTypes.object,
+  flow: PropTypes.object,
+  flowState: PropTypes.object
 }
 
 const TriggerManager = compose(
@@ -422,6 +449,15 @@ const TriggerManager = compose(
 
 // TriggerManager is exported wrapped in FlowProvider to avoid breaking changes.
 const LegacyTriggerManager = props => {
+  // Since the 4.1.0 of cozy-keys-lib, we
+  // render children even if vaultClient is
+  // not defined yet. In that case we we were
+  // displaying TriggerManager without vaultClient.
+  // It was raising an error.
+  // The current fix, is to not display the
+  // TriggerManager when vaultClient is null.
+  const vaultClient = useVaultClient()
+  if (!vaultClient) return null
   const {
     onLaunch,
     onSuccess,

package/src/models/ConnectionFlow.js

@@ -594,7 +594,7 @@ export class ConnectionFlow {
     }
   }
 
-  watchJob(options = {autoSuccessTimer: false}) {
+  watchJob(options = { autoSuccessTimer: false }) {
     this.realtime.subscribe(
       'updated',
       JOBS_DOCTYPE,

package/src/models/ConnectionFlow.spec.js

@@ -8,7 +8,9 @@ import {
   launchTrigger
 } from '../connections/triggers'
 import CozyRealtime from 'cozy-realtime'
-import KonnectorJobWatcher, { watchKonnectorJob } from './konnector/KonnectorJobWatcher'
+import KonnectorJobWatcher, {
+  watchKonnectorJob
+} from './konnector/KonnectorJobWatcher'
 import { konnectorPolicy as biKonnectorPolicy } from '../services/budget-insight'
 import fixtures from '../../test/fixtures'
 import sentryHub from '../sentry'
@@ -118,14 +120,13 @@ describe('ConnectionFlow', () => {
       return flow.getState().running === true
     }
     beforeAll(() => {
-      watchKonnectorJob.mockReturnValue({on: () => ({})})
+      watchKonnectorJob.mockReturnValue({ on: () => ({}) })
     })
 
     afterEach(() => {
       jest.clearAllMocks()
     })
 
-
     it('should render as submitting when there is no account', async () => {
       const { flow } = setup()
       const submitPromise = setupSubmit(flow)
@@ -327,7 +328,7 @@ describe('ConnectionFlow', () => {
   describe('ensureTriggerAndLaunch', () => {
     beforeAll(() => {
       jest.spyOn(cronHelpers, 'fromFrequency').mockReturnValue('0 0 0 * * 0')
-      watchKonnectorJob.mockReturnValue({on: () => ({})})
+      watchKonnectorJob.mockReturnValue({ on: () => ({}) })
     })
 
     afterEach(() => {
@@ -494,7 +495,7 @@ describe('ConnectionFlow', () => {
 
   describe('constructor', () => {
     beforeAll(() => {
-      watchKonnectorJob.mockReturnValue({on: () => ({})})
+      watchKonnectorJob.mockReturnValue({ on: () => ({}) })
     })
 
     afterEach(() => {
@@ -502,8 +503,12 @@ describe('ConnectionFlow', () => {
     })
 
     it('should watch a running trigger', () => {
-      setup({trigger: fixtures.runningTrigger})
-      expect(watchKonnectorJob).toHaveBeenCalledWith(expect.any(Object), {_id: 'runningjobid'}, {autoSuccessTimer: false})
+      setup({ trigger: fixtures.runningTrigger })
+      expect(watchKonnectorJob).toHaveBeenCalledWith(
+        expect.any(Object),
+        { _id: 'runningjobid' },
+        { autoSuccessTimer: false }
+      )
     })
   })
 })