<div dir="ltr"><div>Hi Maik,</div><div><br></div><div>I don't think it has been corrected in master yet : <a href="https://invent.kde.org/graphics/digikam/-/blob/master/core/app/views/sidebar/sidebarwidget.h#L109">https://invent.kde.org/graphics/digikam/-/blob/master/core/app/views/sidebar/sidebarwidget.h#L109</a></div><div>I can fix this in my branch if needed, and it would be included in master when the branches get merged.<br></div><div><br></div><div>Kartik<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jun 25, 2020 at 11:36 AM Maik Qualmann <<a href="mailto:metzpinguin@gmail.com">metzpinguin@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Kartik,<br>
<br>
Yes, that's a typo. It should already be corrected in git/master.<br>
<br>
Maik<br>
<br>
Am Donnerstag, 25. Juni 2020, 07:47:04 CEST schrieb Kartik Ramesh:<br>
> Hello all,<br>
> <br>
> The left sidebar widgets are provided with a notification widget, which<br>
> allows display of slide-in timed notifications. This signal is currently<br>
> named signalNofificationError<br>
> <<a href="https://invent.kde.org/graphics/digikam/-/blob/gsoc20-facesengine-workflow/" rel="noreferrer" target="_blank">https://invent.kde.org/graphics/digikam/-/blob/gsoc20-facesengine-workflow/</a><br>
> core/app/views/sidebar/sidebarwidget.h#L109>, is this a misspell? There<br>
> aren't many places that emit this signal, so correcting it everywhere to<br>
> signalNotificationError wouldn't be much difficult. Should I correct this?<br>
> <br>
> Kartik<br>
<br>
<br>
<br>
<br>
</blockquote></div>