Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/welcome cozy pass web #705

Merged
merged 5 commits into from
Apr 3, 2023
Merged

Feat/welcome cozy pass web #705

merged 5 commits into from
Apr 3, 2023

Conversation

zatteo
Copy link
Contributor

@zatteo zatteo commented Mar 30, 2023

When we open passwords, if no extension is installed, we open cozy-pass-web inside the flagship app to show the onboarding. If extension is installed, we keep calling openAppNative to open native app (or the store).

@zatteo zatteo requested review from acezard and Ldoppea as code owners March 30, 2023 07:28
@Ldoppea
Copy link
Member

Ldoppea commented Mar 31, 2023

nit: For more clarity, instead of extension is installed we should talk about vault is configured. The extension_installed value in database is a legacy from the first CozyPass version.

At that time the only way to access CozyPass was to use the browser extension, so we used that value to check if the extension was installed or not.

Today the extension is not a requirement anymore, so we suggest to install it but the user can skip this step and log from the web interface instead (or from the mobile app). Login to the web is enough to have extension_installed=true

We still use the extension_installed value in our scenario, but the Functional need is to check if the vault is configured, and not only if the extension is installed.

@zatteo zatteo force-pushed the feat/welcome-cozy-pass-web branch 2 times, most recently from 2fbf8a7 to 0973003 Compare March 31, 2023 15:48
@zatteo zatteo closed this Mar 31, 2023
@zatteo zatteo reopened this Mar 31, 2023
@zatteo zatteo force-pushed the feat/welcome-cozy-pass-web branch from 0973003 to c458272 Compare April 1, 2023 10:16
zatteo added 4 commits April 3, 2023 10:32
passwords fetch data with { mode: 'no-cors' } which does not work with
httpServer
- SettingsCollection.get quick fix for io.cozy.settings.bitwarden
- isNativePassInstalledOnDevice type
@zatteo zatteo force-pushed the feat/welcome-cozy-pass-web branch from 9bb7a5b to f206b18 Compare April 3, 2023 08:33
@zatteo zatteo requested review from Crash-- and Ldoppea April 3, 2023 08:33
@zatteo zatteo force-pushed the feat/welcome-cozy-pass-web branch from f206b18 to 3521009 Compare April 3, 2023 08:51
@zatteo zatteo merged commit adf678d into master Apr 3, 2023
@zatteo zatteo deleted the feat/welcome-cozy-pass-web branch April 3, 2023 09:13
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.

3 participants