[GH-ISSUE #1310] Sync interval #873

Closed
opened 2026-02-25 22:38:20 +03:00 by kerem · 6 comments
Owner

Originally created by @kalekad on GitHub (Jan 1, 2023).
Original GitHub issue: https://github.com/floccusaddon/floccus/issues/1310

Which version of floccus are you using?

v4.18.0

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.

Tor Browser 12.0.1(Firefox 102.6esr), Android

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 set the sync interval to 5 minutes and save. I reopen the settings and the sync interval is at 15 minutes.

Expected Behavior

A sync interval of 5 minutes remains stored.

To Reproduce

Debug log provided

  • I have provided a debug log file
Originally created by @kalekad on GitHub (Jan 1, 2023). Original GitHub issue: https://github.com/floccusaddon/floccus/issues/1310 ### Which version of floccus are you using? v4.18.0 ### 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. Tor Browser 12.0.1(Firefox 102.6esr), Android ### 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 set the sync interval to 5 minutes and save. I reopen the settings and the sync interval is at 15 minutes. ### Expected Behavior A sync interval of 5 minutes remains stored. ### To Reproduce - ### Debug log provided - [ ] I have provided a debug log file
kerem 2026-02-25 22:38:20 +03:00
  • closed this issue
  • added the
    bug
    label
Author
Owner

@marcelklehr commented on GitHub (Jan 8, 2023):

It seems only 5minutes was forbidden due to a legacy rule. Fixed now.

<!-- gh-comment-id:1374870537 --> @marcelklehr commented on GitHub (Jan 8, 2023): It seems only 5minutes was forbidden due to a legacy rule. Fixed now.
Author
Owner

@kalekad commented on GitHub (Jan 14, 2023):

In v4.18.1 the color of the indicator changed from red to blue when selecting a 5 min interval, but it still does not save.

<!-- gh-comment-id:1382953750 --> @kalekad commented on GitHub (Jan 14, 2023): In v4.18.1 the color of the indicator changed from red to blue when selecting a 5 min interval, but it still does not save.
Author
Owner

@marcelklehr commented on GitHub (Jan 15, 2023):

Confirmed. Weird. I tested this before the release and it was working :/

<!-- gh-comment-id:1383209540 --> @marcelklehr commented on GitHub (Jan 15, 2023): Confirmed. Weird. I tested this before the release and it was working :/
Author
Owner

@kolinger commented on GitHub (Nov 27, 2023):

Last fix removed one part of condition but identical condition was left few lines bellow: github.com/floccusaddon/floccus@8812859e7b/src/ui/components/OptionSyncInterval.vue (L49-L61)
Where zero index is taken as invalid value and overridden by default 15 minute interval. But zero index is first option of interval and that's 5 minutes, thus 5 minutes is always overridden by 15 minutes.

This is why 5 minute interval still can't be saved with 4.19.1 as #1414 reports and thus this issues wasn't fixed yet.

<!-- gh-comment-id:1827744788 --> @kolinger commented on GitHub (Nov 27, 2023): Last fix removed one part of condition but identical condition was left few lines bellow: https://github.com/floccusaddon/floccus/blob/8812859e7bed4e9000b117ce0a89d6b56ab5c4ab/src/ui/components/OptionSyncInterval.vue#L49-L61 Where zero index is taken as invalid value and overridden by default 15 minute interval. But zero index is first option of interval and that's 5 minutes, thus 5 minutes is always overridden by 15 minutes. This is why 5 minute interval still can't be saved with 4.19.1 as #1414 reports and thus this issues wasn't fixed yet.
Author
Owner

@marcelklehr commented on GitHub (Dec 6, 2023):

Thank you @kolinger for investigating this!

<!-- gh-comment-id:1843530733 --> @marcelklehr commented on GitHub (Dec 6, 2023): Thank you @kolinger for investigating this!
Author
Owner

@github-actions[bot] commented on GitHub (Dec 6, 2024):

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

<!-- gh-comment-id:2521812723 --> @github-actions[bot] commented on GitHub (Dec 6, 2024): This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/floccus#873
No description provided.