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

BSK update for Autofill Script performance improvements #2625

Merged
merged 5 commits into from
Mar 25, 2024

Conversation

amddg44
Copy link
Contributor

@amddg44 amddg44 commented Mar 22, 2024

Task/Issue URL: https://app.asana.com/0/414709148257752/1206900948756760/f
Tech Design URL:
CC:

Description:
Greatly reduces the amount of data passed into the autofill script to improve performance and resolve occasional out of memory crashes

Steps to test this PR:
Lightly smoke test autofill functionality to confirm everything still behaves correctly

  1. Visit https://fill.dev/form/login-simple and save a login
  2. Confirm the Save prompt appears. Tap save and return to the login page
  3. Confirm the key icon icons appear in the login form & that the Fill prompt appears & works correctly
  4. Visit https://fill.dev/form/registration-email and confirm that the dax icon appears in the email field (if you are signed in to email protection) and that the password generation icon appears in the password fields
  5. Tap the generate password icon and confirm a generated password is created & populates the web form

Copy Testing:

  • Use of correct apostrophes in new copy, ie rather than '

Orientation Testing:

  • Portrait
  • Landscape

Device Testing:

  • iPhone SE (1st Gen)
  • iPhone 8
  • iPhone X
  • iPhone 14 Pro
  • iPad

OS Testing:

  • iOS 14
  • iOS 15
  • iOS 16

Theme Testing:

  • Light theme
  • Dark theme

Internal references:

Software Engineering Expectations
Technical Design Template

@github-actions github-actions bot added the bot: not in app board Added by automation for pull requests with tasks not added to iOS App Board Asana project label Mar 22, 2024
Copy link
Contributor

@aataraxiaa aataraxiaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but you need to rebase/merge and remove app board tag

amddg44 added 2 commits March 25, 2024 13:20
…perf-improvements

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
@amddg44 amddg44 removed the bot: not in app board Added by automation for pull requests with tasks not added to iOS App Board Asana project label Mar 25, 2024
amddg44 added a commit to duckduckgo/BrowserServicesKit that referenced this pull request Mar 25, 2024
Task/Issue URL: https://app.asana.com/0/414709148257752/1206900948756760/f
iOS PR: duckduckgo/iOS#2625
macOS PR: duckduckgo/macos-browser#2481
What kind of version bump will this require?: Patch

Description:
Greatly reduces the amount of data passed into the autofill script to improve performance and resolve occasional out of memory crashes
@amddg44 amddg44 merged commit 784fe39 into main Mar 25, 2024
13 checks passed
@amddg44 amddg44 deleted the anya/autofill-script-perf-improvements branch March 25, 2024 13:44
samsymons added a commit that referenced this pull request Mar 25, 2024
# By Anh Do (1) and others
# Via GitHub
* main:
  Fix incorrect omnibar behavior (#2634)
  Replase glitch.me pages with privacy-test-pages.site (#2632)
  BSK update for Autofill Script performance improvements (#2625)
  Send correct platform value for App Store purchase options (#2633)
  Collect extra metadata (#2622)
  Clarified readme "instruments" section (#2607)

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
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.

2 participants