-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
importing from multiple profiles dumps all of the data into the default profile within brave #27591
Labels
enhancement
feature/importer
onboarding
Front-end related to helping a new user learn about features within the browser.
OS/Desktop
priority/P2
A bad problem. We might uplift this to the next planned release.
QA Pass-Linux
QA Pass-macOS
QA Pass-Win64
QA/Test-All-Platforms
QA/Yes
release-notes/include
Milestone
Comments
This was referenced Jan 10, 2023
Verification
Filed:
TestCase 1: Import from Chrome Pre-requisites: Download Chrome and create 3 profiles
Case 1: Single profile from Chrome
Confirmed importing from only one source profile, i.e. Chrome Profile 1, no new Brave profile is createdConfirmed bookmarks, history, extensions and passwords are imported into default profileConfirmed bookmarks are shown on the bookmarks bar and bookmarks sidepanel in a new-tabNote: no cookies are imported
Case 2: Multiple profiles from Chrome
Confirmed three newly created profiles with randomly assigned Avatars are shown in the profile managerConfirmed three newly created profiles have same names from Chrome imports are shown in the profile managerConfirmed data correctly imported into separate profiles
|
step 3 | step 4 | step 6 | step 8 | step 9 | profile manager | imp from chrome P1 | imp from chrome P2 | imp from chrome P5 |
---|---|---|---|---|---|---|---|---|
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
TestCase 2: Import from Firefox
Pre-requisites:
Download Firefox and create 3 profiles
- Default: bookmarks, history, extensions, and passwords
- Profile 2: bookmarks, history, passwords, and extensions
- Profile 3: bookmarks, history, and passwords
- Profile 4-no data: No data
default | Profile 2 | Profile 3 | Profile4-no data |
---|---|---|---|
![]() |
![]() |
![]() |
![]() |
Case 1: Single profile from Firefox
- new profile
- launch Brave
- click `Set Brave as default browser
- select
Mozilla Firefox
- click import
- select
Mozilla Firefox - Profile 2
- click Import Profiles
- click Finish
Confirmed importing from only one source profile, i.e. Firefox Profile 3, no new Brave profile is created
Confirmed bookmarks, history, and passwords are imported into default profile
Confirmed bookmarks are shown in the bookmarks bar and bookmarks sidepanel in a new-tab
extensions are not imported from Firefox as expected and closed this issue - #28492
step 3 | step 4 | step 6 | step 8 | bookmarks | history | passwords |
---|---|---|---|---|---|---|
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
Case 2: Multiple profiles from Firefox
- new profile
- launch Brave
- click `Set Brave as default browser
- select
Mozilla Firefox
- click import
- select
Mozilla Firefox - default
,Mozilla Firefox - Profile 3
,Mozilla Firefox - Profile 4 no data
- click Import Profiles
- click Finish
- click on Profile icon in the new-tab page next to hamburger menu
Confirmed three newly created profiles with randomly assigned Avatars are shown in the profile manager
Confirmed three newly created profiles have same names from Firefox imports are shown in the profile manager
Confirmed data correctly imported into separate profiles Mozilla Firefox - default
, Mozilla Firefox - Profile 2
, and blank profile from Mozilla Firefox - Profile 4 no data
Confirmed bookmarks, history, and passwords are imported into Mozilla Firefox - default
Confirmed bookmarks, history, and passwords are shown in Mozilla Firefox - Profile 3
Confirmed blank profile imported from Mozilla Firefox - Profile 4 no data
- filed #28478
Confirmed Brave Profile 1 is created and shown as expected
step 3 | step 4 | step 6 | step 8 | new-tab | profile manager | default | Profile 3 | Profile 4 |
---|---|---|---|---|---|---|---|---|
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
Verified with
Import from Chrome Pre-requisites: Download Chrome and create 3 profiles
Case 1: Single profile from Chrome - PASSED
Confirmed importing from only one source profile, i.e. Chrome my OG profile, no new Brave profile is createdConfirmed bookmarks, history, extensions and passwords are imported into default profileConfirmed bookmarks are shown on the bookmarks bar and bookmarks sidepanel in a new-tabNote: no cookies are imported
Case 2: Multiple profiles from Chrome - PASSED
Confirmed three newly created profiles with randomly assigned Avatars are shown in the profile managerConfirmed three newly created profiles have same names from Chrome imports are shown in the profile managerConfirmed data correctly imported into separate profiles
|
step 3 | step 4 | step 6 | step 8 | step 9 | profile manager | imp from chrome P1 | imp from chrome P2 | imp from chrome P5 |
---|---|---|---|---|---|---|---|---|
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
enhancement
feature/importer
onboarding
Front-end related to helping a new user learn about features within the browser.
OS/Desktop
priority/P2
A bad problem. We might uplift this to the next planned release.
QA Pass-Linux
QA Pass-macOS
QA Pass-Win64
QA/Test-All-Platforms
QA/Yes
release-notes/include
Description
When running through brave/brave-core#16472 (comment), I noticed that we're importing all the data/extensions into a single profile even though the user might import multiple profiles. We should look into keeping the same user/profile structure when users are importing data from chromium based browsers (not sure if this is possible with Firefox).
For example, in the above case, we should attempt to create a new profile within Brave named
ExtentionTesting
and import all the data into that profile rather than importing it into a single default profile like we do now.Updates from Slack:
Steps to Reproduce
Actual result:
Currently importing data from multiple profiles from other chromium browsers and dumping everything into the default profile.
Expected result:
We should attempt to keep the same structure and create profiles and then import the data into the correct profiles.
Reproduces how often:
100% reproducible using the STR/Cases mentioned above.
Brave version (brave://version info)
Version/Channel Information:
N/A
Yes
Yes
Other Additional Information:
N/A
N/A
N/A
Miscellaneous Information:
The text was updated successfully, but these errors were encountered: