Skip to content

Add refresh ability in autofill#2406

Closed
pavelzw wants to merge 3568 commits intobitwarden:mainfrom
pavelzw:autofill-refresh
Closed

Add refresh ability in autofill#2406
pavelzw wants to merge 3568 commits intobitwarden:mainfrom
pavelzw:autofill-refresh

Conversation

@pavelzw
Copy link

@pavelzw pavelzw commented Mar 7, 2023

Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

Closes #1710

This PR introduces the ability to pull to refresh in the autofill window like in the default app.

Since I am not able to run it in iOS, I didn't try it there... We probably still need to do some iOS specific stuff here... Any help is appreciated to try test it out there.
I tried out #1939 but unfortunately don't have an Apple Developer account.
Works on iOS too.

cc @danstooamerican

Code changes

  • AutofillCiphersPageViewModel.cs: Add RefreshAsync()
  • AutofillCiphersPage.xaml Add RefreshView
  • LoginListViewController.cs Add RefreshControl_ActivatedAsync()
  • LoginListViewController.designer.cs Add RefreshControl_ActivatedAsync
  • iOS.Autofill/MainInterface.storyboard Add refreshControl
  • iOS.Core/Views Add RefreshAsync()

Screenshots

iOS

ios-refresh-view.mov

Android

Bildschirmaufnahme.2023-03-14.um.02.12.06.mov

Before you submit

  • Please check for formatting errors (dotnet format --verify-no-changes) (required)
  • Please add unit tests where it makes sense to do so (encouraged but not required)
  • If this change requires a documentation update - notify the documentation team
  • If this change has particular deployment requirements - notify the DevOps team

eliykat and others added 30 commits June 14, 2022 09:55
…ip enabled (bitwarden#1953)

* fix for missing org items with single org & personal ownership enabled

* fix for ui issue with vault filter state change on pull to refresh
…#1952)

* SG-386 Updated active user when coming from background to the iOS app and the extension had switched users

* Added iOSExtensionActiveUserIdKey to preference keys

* Reorder iOS preference keys
* Pinning the version of NuGet to 5.x

* pinning NuGet verison to 5.9.x

* pinning NuGet to 5.9.0.7134

* pinning NuGet to 5.9.0
… (System) theme (bitwarden#1959)

* support for dark theme selection while using Default (System) theme

* refinements
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…improved the code so there are fewer direct access from the VM to the View (bitwarden#1879)
… service name hardcoded to register/resolve a service (bitwarden#1865)
We only need master password hash because this is currently
only used for sso password setting after auto-provisioning. Key
Connector is not involved in these accounts
…1976)

* Added to AppResources.resx the message warning the user about setting the lockout option to "Never"

* Added the condition to check the newly selected option on the vault timeout settings

* Changed the wording on the warning as to reflect the mobile version

* Changed the vault timeout modal to have the ability to cancel

* Simplified the reversion of value if the user cancels the change
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…gOverlayHelper and also added more logging to see when it happens. (bitwarden#1983)
* feat: update auto totp copy setting

* feat: update show icons settings

* feat: update auto add settings

* feat: update settings and options to sentence case

* feat: update translation keys

With the latest changes the translation keys had diverged from their contents.
This commit fixes that.

* fix: revert AndroidManifest changes

* chore: add todo comments to fix negative functions
* PS-920 - Added feedback to user when saving bad formed URLs
* Added feedback to user when trying to perform login with bad formed URL

* PS-920 - Refactor to use AsyncCommand
*(missing file from previous commit)

* PS-920 - Fixed whitespace formatting

* PS-920 - Removed unused method

* PS-920 - Fixed validation
* Added comment for hard coded string

* PS-920 - Removed unused properties
* Fixed url validations
* Refactored method to local function

* PS-920 - Added exception handling and logging
* Added generic error message string to AppResources
…#1971)

* EC-259 Added Account switching on share extension on iOS, also improved performance for this and exception handling

* EC-259 code formatting

* EC-259 Added account switching to Share extension Send view

* EC-259 Fixed navigation on share extension when a forms page is already presented

* EC-259 Fix send text UI update when going from the iOS extension

* EC-259 Improved DateTimeViewModel with helper property to easily setup date and time at the same time and applied on usage
Added Opera GX Support for autofill
michalchecinski and others added 18 commits February 14, 2023 10:10
* Add calculating hashes for apk files

* Install checksum

* Fix name of artifact

* Fix

* Add SHA files to github release

* Update .github/workflows/build.yml

Co-authored-by: Vince Grassia <593223+vgrassia@users.noreply.github.com>

* Update .github/workflows/build.yml

Co-authored-by: Vince Grassia <593223+vgrassia@users.noreply.github.com>

* Update .github/workflows/release.yml

Co-authored-by: Vince Grassia <593223+vgrassia@users.noreply.github.com>

---------

Co-authored-by: Vince Grassia <593223+vgrassia@users.noreply.github.com>
Co-authored-by: bitwarden-devops-bot <106330231+bitwarden-devops-bot@users.noreply.github.com>
* [PS-2280] Retain app settings on logout

* adjustments
* [SG-1057] Make check breaches true by default (bitwarden#2351)

(cherry picked from commit 6875389)

* [SG-1055] Init strength bar color (bitwarden#2352)

(cherry picked from commit 3780587)

* [SG-1052] Fix copy (bitwarden#2353)

(cherry picked from commit 0bd1b3f)

* [SG-1056] Fix login prompt loop for added accounts (bitwarden#2357)

* SG-1056 Fix login prompt loop for added accounts

* SG-1056 Fix PR comment

(cherry picked from commit f42c677)

* Bumped version to 2023.2.0

---------

Co-authored-by: André Bispo <abispo@bitwarden.com>
Co-authored-by: Carlos Gonçalves <cgoncalves@bitwarden.com>
Co-authored-by: bitwarden-devops-bot <106330231+bitwarden-devops-bot@users.noreply.github.com>
Co-authored-by: Opeyemi Alao <54288773+Eeebru@users.noreply.github.com>
Co-authored-by: bitwarden-devops-bot <106330231+bitwarden-devops-bot@users.noreply.github.com>
* work: adjusted numbers to 12

* work: made string dynamic

* fix: spaces
Co-authored-by: bitwarden-devops-bot <106330231+bitwarden-devops-bot@users.noreply.github.com>
* EC-763  Implemented watchOS parts on cake build

* EC-763 Improved setting the version name of the watch by caching the version on when updating the ios info plist file

* EC-763 Fix cake build criteria to use lambda version so that it takes into consideration that the field might change on executing the script

* EC-763 Added iOS and watchOS icons update to the cake build script
)

* Restore copy notification toast on Android 13+

* fixed space
Co-authored-by: bitwarden-devops-bot <106330231+bitwarden-devops-bot@users.noreply.github.com>
* EC-1002 BEEEP Added ability to change language in app

* EC-1002 fix format

* EC-1002 Renamed IPreferencesStorageService to ISynchronousStorageService

* EC-1002 Moved get/set Locale to the StateService and added the StorageMediatorService to a new way to interact with the storage. Later the StateService will only interact with this mediator instead of directly with the storage services, with this we have more control inside the mediator and we can have both sync and async methods to interact with storages handled by the mediator
* Update iOS Distribution cert and provision profiles

* Rename the provision profiles

* Update the App Store provision profile names in plist

* Update Watch provision profile

* Remove testing code in pipeline

* Remove more test code
Co-authored-by: bitwarden-devops-bot <106330231+bitwarden-devops-bot@users.noreply.github.com>
Co-authored-by: Daniel Augustin <daniel.augustin1@gmx.de>
@CLAassistant
Copy link

CLAassistant commented Mar 7, 2023

CLA assistant check
All committers have signed the CLA.

danstooamerican and others added 4 commits March 14, 2023 00:28
Co-authored-by: Pavel Zwerschke <pavelzw@gmail.com>
Co-authored-by: Daniel Augustin <daniel.augustin1@gmx.de>
@pavelzw pavelzw changed the title Add reload ability in autofill Add refresh ability in autofill Mar 14, 2023
@Aviator7595
Copy link

Aviator7595 commented Apr 7, 2024

Hi @pavelzw
I'm very intrested in this feature. Would it be possible to resolve the conflicts and merge the PR?
Thanks in advance

@pavelzw
Copy link
Author

pavelzw commented Apr 8, 2024

I don't have merging rights here. If the maintainers are open for this change, I could try to fix the merge conflicts.

@Aviator7595
Copy link

Thanks pavelzw

Hi @fedemkr
I noticed that you have been recently reviewing and merging several PR on both ios and android clients.
Do you mind having a look on this one please? I do beleive that this is an important feature as I'm personally very annoyed by the fact that autofill window is not sync.

Thanks in advance

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.

No refresh before autofilling