Skip to content

263: Move items between collections#267

Draft
ZanderCowboy wants to merge 11 commits intodevelopfrom
263-add-functionality-to-move-items-between-collections
Draft

263: Move items between collections#267
ZanderCowboy wants to merge 11 commits intodevelopfrom
263-add-functionality-to-move-items-between-collections

Conversation

@ZanderCowboy
Copy link
Copy Markdown
Owner

@ZanderCowboy ZanderCowboy commented Feb 13, 2026

#263

Summary

  • Adds support for moving an entry from one collection (tab) to another.
  • Updates HomeBloc and EntryRepository to persist entry moves.
  • Includes UI plumbing for drag/drop + reorder in the home layout.

Details

  • Core change is IEntryRepository.moveEntryToTab(...) with an implementation in EntryRepository.
  • Home flow exposes an event to move an entry between tabs and updates state optimistically.

Test plan

  • melos analyze
  • melos test:core

Notes / risks

@sonarqubecloud
Copy link
Copy Markdown

Bring PR #267 up to date with the latest develop branch.

Made-with: Cursor
@ZanderCowboy ZanderCowboy changed the title 263 add functionality to move items between collections 263: Move items between collections Apr 26, 2026
Restore TabLayout part structure, revert vertical home conflict artifacts, and fix HomeBloc moveEntryToTab test stubs.

Made-with: Cursor
EntryCard now requires isLayoutVertical; TabPlaceholderCard reads it from AppLayout.

Made-with: Cursor
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 26, 2026

Codecov Report

❌ Patch coverage is 91.48936% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.60%. Comparing base (93bd56d) to head (5172cc1).
⚠️ Report is 19 commits behind head on develop.

Files with missing lines Patch % Lines
...itories/implementation/entry/entry_repository.dart 88.88% 2 Missing ⚠️
...kages/core/lib/src/application/home/home_bloc.dart 96.42% 1 Missing ⚠️
...ages/core/lib/src/application/home/home_event.dart 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #267      +/-   ##
===========================================
+ Coverage    82.19%   83.60%   +1.40%     
===========================================
  Files           49       49              
  Lines         1556     1580      +24     
===========================================
+ Hits          1279     1321      +42     
+ Misses         277      259      -18     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ZanderCowboy ZanderCowboy added the feature Create a new feature label Apr 26, 2026
@ZanderCowboy ZanderCowboy moved this to In Progress in Multichoice Apr 26, 2026
@ZanderCowboy ZanderCowboy linked an issue Apr 26, 2026 that may be closed by this pull request
ZanderCowboy and others added 3 commits April 26, 2026 15:54
@sonarqubecloud
Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Create a new feature

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

Add functionality to move items between collections

1 participant