Skip to content
This repository was archived by the owner on Feb 24, 2025. It is now read-only.

Conversation

@SabrinaTardio
Copy link
Collaborator

@SabrinaTardio SabrinaTardio commented May 5, 2023

Task/Issue URL: https://app.asana.com/0/1203822806345703/1204501137370799/f

Description: Adds export logins to Autofill 3 dots menu and login export button to autofill settings

Steps to test this PR:

  1. Go to the autofill settings. Check that there is an export logins button. Press on it and check the correct dialog is shown
  2. Go to the browser 3 dots menu click on autofill. On the autofill 3 dots menu there should be an export options and under that an export logins and export bookmarks option. Check each present the expected dialog.
  3. Go to Bookmarks on the main menu check that there is an export bookmarks button. Press on it and check the correct dialog is shown
  4. Go to Bookmarks on the 3 dots menu check that there is an export bookmarks button. Press on it and check the correct dialog is shown
  5. Check export menus are disabled when no bookmarks / passwords

<!—
Tagging instructions
If this PR isn't ready to be merged for whatever reason it should be marked with the DO NOT MERGE label (particularly if it's a draft)
If it's pending Product Review/PFR, please add the Pending Product Review label.

If at any point it isn't actively being worked on/ready for review/otherwise moving forward (besides the above PR/PFR exception) strongly consider closing it (or not opening it in the first place). If you decide not to close it, make sure it's labelled to make it clear the PRs state and comment with more information.
—>

Internal references:

Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

Copy link
Contributor

@afterxleep afterxleep left a comment

Choose a reason for hiding this comment

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

Neat!. :shipit:

Screenshot 2023-05-05 at 3 51 35 PM

@SabrinaTardio SabrinaTardio requested a review from afterxleep May 9, 2023 10:42
@SabrinaTardio SabrinaTardio requested a review from afterxleep May 15, 2023 08:36
@SabrinaTardio SabrinaTardio merged commit 24735f3 into develop May 19, 2023
@SabrinaTardio SabrinaTardio deleted the sabrina/Add-Login-export-links-to-settings-and-Autofill-menu branch May 19, 2023 10:05
samsymons added a commit that referenced this pull request May 23, 2023
# By Sabrina Tardio (2) and Sam Symons (1)
# Via GitHub
* upstream_develop:
  Check that the source and target frame security origins are equal (#1207)
  Add activation points pixels (#1206)
  Add Login export links to settings and Autofill 3 dot menu (#1183)

# Conflicts:
#	DuckDuckGo/Menus/MainMenuActions.swift
#	DuckDuckGo/Statistics/PixelEvent.swift
#	DuckDuckGo/Statistics/PixelParameters.swift
#	UnitTests/Menus/Mocks/CapturingOptionsButtonMenuDelegate.swift
samsymons added a commit that referenced this pull request May 27, 2023
* release/1.42.0: (26 commits)
  Bump version to 1.42.0 (28)
  Update embedded files
  Network Protection (#1211)
  Update BSK with autofill 7.1.0 (#1225)
  Autofill password generation support for iOS (#1212)
  Sync Engine with support for syncing bookmarks (#1203)
  Bump BSK for messaging updates (#1213)
  add newTab pixel, fire pixels only for new users first time (#1219)
  Update tds endpoint (#1218)
  prevent devices from being fetch when sync not visible (#1214)
  License and contributor guidelines for open-sourcing (#1189)
  Add a unique Bitwarden decryption failure pixel (#1197)
  Bump version to 1.41.0 (27)
  Update embedded files
  Check that the source and target frame security origins are equal (#1207)
  Add activation points pixels (#1206)
  Add Login export links to settings and Autofill 3 dot menu (#1183)
  sync device connected names (#1205)
  Autofill/subdomain matching (#1122)
  add-windows-browser-download-link-privacy-config (#1198)
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants