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

Autofill letter icons for macOS #1911

Merged
merged 5 commits into from
Aug 16, 2023
Merged

Autofill letter icons for macOS #1911

merged 5 commits into from
Aug 16, 2023

Conversation

afterxleep
Copy link
Collaborator

@afterxleep afterxleep commented Aug 11, 2023

Task/Issue URL: https://app.asana.com/0/1204099484721401/1202300800602708/f

Description:
Removes WebsiteAccount extension and moves to BSK, so it can be used on Mac. This PR does not include any changes for iOS

Steps to test this PR:

  1. Smoke test Logins UI, check Letter icons are generated when you add a login
Screenshot 2023-08-11 at 4 44 11 PM
Internal references:

Software Engineering Expectations
Technical Design Template

@afterxleep afterxleep changed the title Move SecureVault.WebsiteAccount extension to BSK Autofill letter icons for macOS Aug 11, 2023
@afterxleep afterxleep marked this pull request as ready for review August 11, 2023 14:52
# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
Copy link
Contributor

@amddg44 amddg44 left a comment

Choose a reason for hiding this comment

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

This wouldn't compile for me without removing WebsiteAccountExtension.swift from Build Phases > Compile Sources & unit tests are failing for the same reason - perhaps you missed a commit? Otherwise looks good!

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
#	DuckDuckGo/AutofillLoginDetailsViewModel.swift
#	DuckDuckGo/AutofillLoginListItemViewModel.swift
#	DuckDuckGo/WebsiteAccountExtension.swift
afterxleep added a commit to duckduckgo/BrowserServicesKit that referenced this pull request Aug 16, 2023
Required:
Task/Issue URL: https://app.asana.com/0/1204099484721401/1202300800602708/f
iOS PR: duckduckgo/iOS#1911
macOS PR: duckduckgo/macos-browser#1475
What kind of version bump will this require?: Minor

Description:
Moves WebsiteAccount methods from iOS to BSK to be used on macOS
@afterxleep afterxleep merged commit 6d4e2b0 into develop Aug 16, 2023
6 checks passed
@afterxleep afterxleep deleted the daniel/autofill-icons branch August 16, 2023 15:34
samsymons added a commit that referenced this pull request Aug 17, 2023
# By Daniel Bernal (1) and others
# Via GitHub
* develop:
  BSK Hotfix 74.0.1-1 (#1924)
  Post merge PR check (#1920)
  Autofill letter icons for macOS (#1911)

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
samsymons added a commit that referenced this pull request Aug 24, 2023
* develop: (84 commits)
  Improve Sync-related database cleaning logic (#1933)
  Fix PacketTunnelProvider embedding for Alpha builds (#1932)
  Add alpha specific app groups (#1934)
  Replaces useSystemKeychain with keychain type (#1928)
  Add Commit hook install script (#1845)
  Run checks on all PRs (#1929)
  extend valid urls to include IPv6 (#1919)
  Status view error UI (#1915)
  NetP Design Review minus errors (#1923)
  Add a GHA job to build the app in release configuration (#1848)
  Release 7.85.0 (#1926)
  Update WireGuard version (#1918)
  BSK Hotfix 74.0.1-1 (#1924)
  Post merge PR check (#1920)
  Autofill letter icons for macOS (#1911)
  Alpha build readiness (#1913)
  bump C-S-S to 4.32.0 (#1914)
  Update NetP endpoint (#1905)
  Network Protection debug menu (#1910)
  Add support for syncing Credentials (#1844)
  ...
@amddg44 amddg44 mentioned this pull request Sep 27, 2023
13 tasks
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