floccusaddon / floccus

:cloud: Sync your bookmarks privately across browsers and devices
https://floccus.org
Mozilla Public License 2.0
5.6k stars 237 forks source link

Manually adding bookmark android app throws error #1725

Open whisperdancer opened 3 days ago

whisperdancer commented 3 days ago

Which version of floccus are you using?

5.2.7

How many bookmarks do you have, roughly?

500

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

No

Sync method

Nextcloud Bookmarks

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

Firefox 130.0.1

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

Latest

Which version of Nextcloud? (if relevant)

29.x

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

No response

Describe the Bug

I tried adding a bookmark via the flocus android app and received an error "Failed to map parent Id" It then added a lock and unable to sync until the app eventually removed the locks.

Expected Behavior

I expected to be able to add a bookmark using android app and not receive an error.

To Reproduce

  1. Manually add bookmark using android app. Add it one or 2 levels down the bookmark tree.
  2. Get error "failed to map parentId"
  3. Now all clients are stuck in a "Scheduled" mode and won't sync until some time has passed and flocus removes the locks.

Debug log provided

github-actions[bot] commented 3 days ago

Hello :wave:

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 a few years ago, 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! To continue the development and maintenance of this project in a sustainable way it is expected that you donate to the project when opening a ticket, if you're not a donor already. You can find donation options at https://floccus.org/donate/. Thank you!

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 Nextcloud 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 :blue_heart:

whisperdancer commented 3 days ago

I just found this page which discusses my issue. https://floccus.org/faq/

I will wait 2 hours and report back if it has started syncing again. Note, I removed the android app and cleared cache. If all goes well with the browser, then I will install android app again and I believe it should sync ok.

Thanks

whisperdancer commented 3 days ago

Update, flocus removed the locks and the issue is resolved. Syncing great again. However, the issue with adding a manual bookmark via the android app which caused the issue remains.

marcelklehr commented 1 day ago

You can get around the failed to map parentId error by triggering a sync from scratch in the profile settings. Make sure to have a backup of your bookmarks before you do this.