Don't remove tasks from the inbox when associating them to a context.
ClosedPublic
Actions

Authored by dfaure on Oct 3 2016, 9:43 PM.

Details

Reviewers

ervin
franckarrecot

Commits

R4:91152a049546: Don't remove tasks from the inbox when associating them to a context.

Summary

Otherwise the user has to go through all projects *and* all contexts
for the daily task review. With this change, going through all projects
(and the inbox, to sort tasks into projects) is enough.

The fix involves a lot more kiwis :-)

Test Plan

make test + manual testing

Diff Detail

Repository

R4 Zanshin

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

dfaure updated this revision to Diff 7067.Oct 3 2016, 9:43 PM

dfaure retitled this revision from to Don't remove tasks from the inbox when associating them to a context..

dfaure updated this object.

dfaure edited the test plan for this revision. (Show Details)

dfaure added a reviewer: ervin.

dfaure added a subscriber: Zanshin.

seems good, even though phabricator review board indentation looks a bit buggy.

This revision is now accepted and ready to land.Oct 8 2016, 8:21 PM

dfaure closed this revision.Oct 12 2016, 4:33 AM

Revision Contents
Changeset List

			Path	Packages
M			src/akonadi/akonaditaskqueries.cpp (3 lines)
M			tests/features/zanshin/features/datasource/datasource-selection.feature (1 line)
M			tests/features/zanshin/features/inbox/inbox-display.feature (1 line)
M			tests/features/zanshin/features/inbox/inbox-drag-and-drop.feature (7 lines)
M			tests/units/akonadi/akonaditaskqueriestest.cpp (28 lines)

Commit	Tree	Parents	Author	Summary	Date
df1f60f19faf	ccb6fb77ff69	d45b3d1da73a	David Faure	Don't remove tasks from the inbox when associating them to a context. (Show More…)	Oct 3 2016, 9:07 PM