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

E029 Failsafe #1796

Closed
1 task
sircode opened this issue Dec 14, 2024 · 9 comments
Closed
1 task

E029 Failsafe #1796

sircode opened this issue Dec 14, 2024 · 9 comments
Labels

Comments

@sircode
Copy link

sircode commented Dec 14, 2024

Which version of floccus are you using?

latest

How many bookmarks do you have, roughly?

tested 20

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

No

Sync method

WebDAV

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

No response

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

Added bookmarks to Android app. But Sync want to delete them although sync settings are set to "merge"
Screenshot_2024-12-11-14-47-41

Expected Behavior

Do not delete new bookmarks

To Reproduce

as written

Debug log provided

  • I have provided a debug log file
@sircode sircode added the bug label Dec 14, 2024
Copy link

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 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 💙

@sircode sircode changed the title E=29 Failsafe E029 Failsafe Dec 14, 2024
@marcelklehr
Copy link
Member

So this is the first sync ever? That's curious.

@sircode
Copy link
Author

sircode commented Dec 14, 2024

No, not the first sync.
Webserver and Android App were synced. Then I added entries in the app. After that the sync failed as described

@sircode
Copy link
Author

sircode commented Dec 15, 2024

This error may also be caused by testing sync with both version xbel and html.
When editing with xbel settings, then switching to html, syncing html fails.
When editing with html settings, then switching to xbel , syncing xbel fails.

@marcelklehr
Copy link
Member

marcelklehr commented Dec 15, 2024

Aha, i see. I can reproduce this 🎉

@marcelklehr
Copy link
Member

marcelklehr commented Dec 15, 2024

When editing with xbel settings, then switching to html, syncing html fails.

You shouldn't expect to be able to sync to a HTML file when XBEL is set in your profile, though. How should that even work?

marcelklehr added a commit that referenced this issue Dec 15, 2024
@github-project-automation github-project-automation bot moved this to Backlog in Floccus Dec 15, 2024
@marcelklehr marcelklehr moved this from Backlog to Triaging in Floccus Dec 15, 2024
@sircode
Copy link
Author

sircode commented Dec 15, 2024

Not really. I was expecting sync as bookmarks.xbel only, when xbel is selected in settings.
Or sync of bookmarks.html when html is selected in settings.
Both would we luxury ;)

General question. xbel and html seem to handle separate data in the app?

I thought the data are the same and one can just change the format. But that does not seem to be the case.

Screenshot_2024-12-15-19-32-01.jpg

Screenshot_2024-12-15-19-32-37.jpg

@marcelklehr
Copy link
Member

The data is in the file that you specify, so when you change the file, the data changes as well.

@marcelklehr
Copy link
Member

I'm closing this as this seems solved now that we've cleared this up :)

@github-project-automation github-project-automation bot moved this from Triaging to Done in Floccus Dec 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

No branches or pull requests

2 participants