KTp plasmoid repostiories

[Daniel Vrátil]

On Monday 07 of January 2013 16:36:25 David Edmundson wrote:
> Review:
> 
>  - What DrDanz said about keeping history...
> I'm personally not too fussed, but it probably is the right thing to
> do. Please ask if you need some git help. It can be confusing - but
> we've done it before moving stuff.

Almost done, I've created new branches with full history, I will merge applets 
tonight and ask a sysadmin to disable hooks temporarily before pushing.

>  - There's generic declarative code sneaking into the bitbucket
> ktp-active repo. Catch that now, and put it in here.
> (it's also clearly copy and pasted from presence-applet, as it ref
> counts itself.. which makes no sense in the active class.)

Will coordinate with mentero (after the branches are published).

> 
>  - If you use this plugin in it's entireity you get two sets of
> Account/ContactFactories.
> Now this is all in the same plugin, I think it's very important to fix
> this. This can be done after merge, but must not be forgotten about.
> You will end up in a situation with code not working in a really
> confusing way. We need a generic system that work's with : ktp-active
> (inc the presence class being used there), and apol's proposed patch.

I'm not sure I understand here - the problem is that there are multiple 
components constructing Factories, so using some of them in a single app 
breaks things?

> 
>  - Rename hidewindowcomponent.cpp to use-hyphens.

Done

Dan

> 
> Dave
> _______________________________________________
> KDE-Telepathy mailing list
> KDE-Telepathy at kde.org
> https://mail.kde.org/mailman/listinfo/kde-telepathy
-- 
dvratil at redhat.com | Associate Software Engineer / BaseOS / KDE, Qt
GPG Key: 0xC59D614F6F4AE348
Fingerprint: 4EC1 86E3 C54E 0B39 5FDD B5FB C59D 614F 6F4A E348
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 490 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20130107/d71505a1/attachment.sig>