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

Browser Built-in folders not mapped correctly on Avast Browser #1694

Open
1 task
zzzatang opened this issue Aug 9, 2024 · 6 comments
Open
1 task

Browser Built-in folders not mapped correctly on Avast Browser #1694

zzzatang opened this issue Aug 9, 2024 · 6 comments

Comments

@zzzatang
Copy link

zzzatang commented Aug 9, 2024

Which version of floccus are you using?

5.2.5

How many bookmarks do you have, roughly?

900

Are you using other means to sync bookmarks in parallel to floccus?

No

Sync method

Google Drive

Which browser are you using? In case you are using the phone App, specify the Android or iOS version and device please.

Chrome Version 127.0.6533.100 (Official Build) (64-bit)

Which version of Nextcloud Bookmarks are you using? (if relevant)

No response

Which version of Nextcloud? (if relevant)

No response

What kind of WebDAV server are you using? (if relevant)

No response

Describe the Bug

I am a new user.
I tried to switch over to Floccus today and faced issues.

I set-up a profile (two attempts) at root level but Floccuss failed to sync the bookmarks from the Other folder across to all devices / browsers

Expected Behavior

Given the profile was created at Root level, I was expecting all bookmarks to be sync across all browsers

To Reproduce

I have tried the following twice:
1 Create a new profile at ROOT LEVEL in Chrome
2. Export profile to a Dropbox folder, so it syncs across all PC's / laptops
3. Upload all bookmarks,
4. Open Avast browser on the same Laptop and Open Chrome on another Laptop,
5. Delete all existing bookmarks in the browsers opened in step 4
6. Add the profile from Dropbox folder to the two browsers opened in step 4
7. Pull bookmarks using Floccus.
8. Check bookmarks downloaded by Floccus. In both browsers on the bookmarks in the bookmarks toolbar were downloaded. BUT The Other bookmarks remains empty
Both laptops all three browsers were fully updated this morning before I attempted to switch over to Floccus.

Debug log provided

  • I have provided a debug log file
@zzzatang zzzatang added the bug label Aug 9, 2024
Copy link

github-actions bot commented Aug 9, 2024

Hello 👋

Thank you for taking the time to open this issue with floccus. I know it's frustrating when software
causes problems. You have made the right choice to come here and open an issue to make sure your problem gets looked at
and if possible solved.
I'm Marcel and I created floccus and have been maintaining it ever since.
I currently work for Nextcloud which leaves me with less time for side projects like this one
than I used to have.
I still try to answer all issues and if possible fix all bugs here, but it sometimes takes a while until I get to it.
Until then, please be patient.
Note also that GitHub is a place where people meet to make software better together. Nobody here is under any obligation
to help you, solve your problems or deliver on any expectations or demands you may have, but if enough people come together we can
collaborate to make this software better. For everyone.
Thus, if you can, you could also have a look at other issues to see whether you can help other people with your knowledge
and experience. If you have coding experience it would also be awesome if you could step up to dive into the code and
try to fix the odd bug yourself. Everyone will be thankful for extra helping hands!
One last word: If you feel, at any point, like you need to vent, this is not the place for it; you can go to the forum,
to twitter or somewhere else. But this is a technical issue tracker, so please make sure to
focus on the tech and keep your opinions to yourself.

I look forward to working with you on this issue
Cheers 💙

@marcelklehr
Copy link
Member

Hey @zzzatang
You may need to set the folders to sync again after importing the profile, because the IDs are likely not valid

@marcelklehr
Copy link
Member

Also, floccus is not tested this in Avast Browser, so there might be a difference in how it represents the built-in bookmarks folders. Currently this is only tested in Edge, Chrome Vivaldi, Brave and Firefox. If setting the sync folder manually after importing the profile does not help, the quick fix is to setup one profile for the bookmarks bar and one profile for "Other bookmarks". In the meanwhile I can try to add support for root folder sync in Avast Browser.

@github-project-automation github-project-automation bot moved this to Backlog in Floccus Aug 9, 2024
@marcelklehr marcelklehr moved this from Backlog to Triaging in Floccus Aug 9, 2024
@zzzatang
Copy link
Author

zzzatang commented Aug 9, 2024

Hi @marcelklehr Thanks for your quick response.

I suspect the main issue could be folder mapping when syncing at Root level and also when exporting / importing two profiles in one Json file export. I am not sure if there was any other issue.

I did the following as a workaround, seems to have worked:

  1. Create two separate profiles for the Bookmarks Bar (toolbar) and Bookmarks Menu (Other Bookmarks)
  2. Export two profiles as two separate json files. (earlier I had exported the two together, I am not sure if that had caused any issues, I was just being too cautious)
  3. Import the two profile files one at a time and set the folder mapping correctly for each browser Edge, Firefox, Chrome and Avast across the two laptops
  4. Force download from the mastercopy for each profile and then set it to auto-sync.

@zzzatang
Copy link
Author

zzzatang commented Aug 9, 2024

It would be great if you could add mapping when sync'ing at ROOT level for various browsers.

E.g. 1. Bookmarks Menu (Firefox) <-> Other Bookmarks (Chrome) <-> Other Favourites (Edge)
2. Bookmarks Toolbar (Firefox) <-> Bookmarks Bar (Chrome) <-> Favourites bar (Edge)

Many times the folder mapping had failed when I imported the profiles exported from the Chrome browser.

It might be helpful if the import process asks to confirm the target mapping folder or is able to automatically map it depending on the browser

@marcelklehr
Copy link
Member

So, there's two different things involved here. When syncing the root folder, the built-in folders should be mapped correctly to the built-in folders from other browsers. This is already implemented for Chrome and Firefox. If this doesn't work for Avast browser, it would be useful to get a log from floccus on that browser.

Mapping built-in folders on import is not implemented yet as right now users will always have to reset the folders after import (there's even a note in the UI about this).

@marcelklehr marcelklehr changed the title Root Level sync only syncs the Bookmarks Toolbar (not the other Bookmarks folder) Browser Built-in folders not mapped correctly on Avast Browser Sep 6, 2024
@marcelklehr marcelklehr moved this from Triaging to To Do: Bugs in Floccus Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: To Do: Bugs
Development

No branches or pull requests

2 participants