Store contexts as todos, not tags, step 2: port ContextRepository
ClosedPublic

Authored by dfaure on Feb 19 2019, 12:02 AM.

Details

Summary

Port Akonadi::ContextRepository to the new serializer methods.
This required passing a DataSource to create(), which has impact all the
way up to the GUI, which now needs to request a data source from the
user when creating a context.

Diff Detail

Repository
R4 Zanshin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
dfaure requested review of this revision.Feb 19 2019, 12:02 AM
dfaure created this revision.
ervin accepted this revision.Mar 2 2019, 9:53 AM

Couples of comments, but they are for different commits, they don't need to be done in this commit.

src/widgets/messageboxinterface.h
45

I'd indeed get rid of it if it's unused.

src/widgets/newprojectdialog.h
52

That asks for renaming that class to "NewPageDialog" BTW, likely in a different commit though.

src/widgets/newprojectdialoginterface.h
36

See comment above indeed ;-)

This revision is now accepted and ready to land.Mar 2 2019, 9:53 AM
dfaure added inline comments.Mar 2 2019, 11:31 AM
src/widgets/newprojectdialog.h
52

I agree on renaming, but I'm not sure about that name. What if one day we add a different kind of page?

How about NameAndDataSourceDialog? That's what it really is.

This revision was automatically updated to reflect the committed changes.
ervin added a comment.Mar 2 2019, 3:23 PM

NameAndDataSourceDialog it is then.