[PR #1076] [MERGED] Fix REORDER processing #1812

New issue

Closed

opened 2026-02-25 23:30:54 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 23:30:54 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/floccusaddon/floccus/pull/1076
Author: @marcelklehr
Created: 3/14/2022
Status: ✅ Merged
Merged: 3/15/2022
Merged by: @marcelklehr

Base: develop ← Head: fix/ordering

📝 Commits (6)

c3edb91 BrowserController#updateBadge: Make syncing have higher precedence than error
7bcbd51 Tests: Check order
36554a6 Default: removeItemFromReorders instead of avoidTargetReorders
c00d6ff Fix expected order in test: should not delete additions while sync is running
af81344 Fix expected order in test: should not fail when moving both folders and contents
ec73560 Merge: Allow REORDERs

📊 Changes

4 files changed (+72 additions, -65 deletions)

View changed files

📝 src/lib/browser/BrowserController.js (+3 -3)
📝 src/lib/strategies/Default.ts (+15 -7)
📝 src/lib/strategies/Merge.ts (+0 -4)
📝 src/test/test.js (+54 -51)

📄 Description

No description provided

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/floccusaddon/floccus/pull/1076 **Author:** [@marcelklehr](https://github.com/marcelklehr) **Created:** 3/14/2022 **Status:** ✅ Merged **Merged:** 3/15/2022 **Merged by:** [@marcelklehr](https://github.com/marcelklehr) **Base:** `develop` ← **Head:** `fix/ordering` --- ### 📝 Commits (6) - [`c3edb91`](https://github.com/floccusaddon/floccus/commit/c3edb91e17cd2aa22f8ec642c2f7b95f046a6fc2) BrowserController#updateBadge: Make syncing have higher precedence than error - [`7bcbd51`](https://github.com/floccusaddon/floccus/commit/7bcbd511e57c78f3cd28a85422ee290e25c8d063) Tests: Check order - [`36554a6`](https://github.com/floccusaddon/floccus/commit/36554a6294a93477117e47c67f9bca40f916d35e) Default: removeItemFromReorders instead of avoidTargetReorders - [`c00d6ff`](https://github.com/floccusaddon/floccus/commit/c00d6ff5080abc425d68ca0fbe7d9fc022f353f0) Fix expected order in test: should not delete additions while sync is running - [`af81344`](https://github.com/floccusaddon/floccus/commit/af8134466cb3d87249bd115978d7820cc4994335) Fix expected order in test: should not fail when moving both folders and contents - [`ec73560`](https://github.com/floccusaddon/floccus/commit/ec735600822be8f8931ade4d559d45a6cc5a907c) Merge: Allow REORDERs ### 📊 Changes **4 files changed** (+72 additions, -65 deletions) <details> <summary>View changed files</summary> 📝 `src/lib/browser/BrowserController.js` (+3 -3) 📝 `src/lib/strategies/Default.ts` (+15 -7) 📝 `src/lib/strategies/Merge.ts` (+0 -4) 📝 `src/test/test.js` (+54 -51) </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-02-25 23:30:54 +03:00

closed this issue
added the
pull-request
label

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#1812

No description provided.

Rows
Columns