[Kde-pim] Review Request: Give the user some hint that an unexisting resource is configured

[Jason Kasper]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/787/#review1254
-----------------------------------------------------------



Screenshot: Message shown when the configured resource does not exist
<http://reviewboard.kde.org//r/787/#scomment33>
    Hm. Sorry I didn't catch this before, but could you change this to:
    
    The previous Akonadi collection is invalid. You must select a valid Akonadi collection above and click apply before you can use this conduit.
    
    My reasoning here is that the user will not have any idea what collection they had chosen, or even what it means (since they don't see "Collection 20" anywhere in their Akonadi configuration settings), and it will only confuse them. I also think that we should show this error or the warning below, but not both. This one and the one below it say basically the same thing, don't they?

- Jason


On 2009-06-01 13:28:02, Bertjan Broeksema wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/787/
> -----------------------------------------------------------
> 
> (Updated 2009-06-01 13:28:02)
> 
> 
> Review request for KDE PIM, Douglas Harms and Jason Kasper.
> 
> 
> Summary
> -------
> 
> It seems to happen so now and than that people configure an Akonadi resource, sync with KPilot and than delete/create a new resource for the same file. The problem is that KPilot cannot know that it is the same file and the user *must* therefore reconfigure the conduit. This patch gives the user some hints about the fact that the configuration currently contains a resource that doesn't exist (anymore).
> 
> 
> This addresses bug 192187.
>     https://bugs.kde.org/show_bug.cgi?id=192187
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdepim/kpilot/conduits/akonadibase/akonadi-setup-widget.ui 976457 
>   trunk/KDE/kdepim/kpilot/conduits/akonadibase/akonadisetupwidget.cc 976457 
> 
> Diff: http://reviewboard.kde.org/r/787/diff
> 
> 
> Testing
> -------
> 
> 
> Screenshots
> -----------
> 
> Message shown when the configured resource does not exist
>   http://reviewboard.kde.org/r/787/s/126/
> 
> 
> Thanks,
> 
> Bertjan
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/