<div dir="ltr"><div>Hi Felipe, there doesn't seem to have been any movement on this for some months, are you still interested in working on it?  If it's a useful app it can be added to KDE gear to get releases done for example.<br></div><div><br></div><div>Jonathan</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, 3 Sept 2022 at 10:40, Tomaz Canabrava <<a href="mailto:tcanabrava@kde.org">tcanabrava@kde.org</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"><div dir="ltr"><div>Hello All,</div><div><br></div><div>All discussion was resolved by Felipe in the past few weeks,</div><div>I Assume this is done, and should be moved out of KDE Review?</div><div><br></div><div>Best.</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jun 28, 2022 at 4:54 PM Felipe Kinoshita <<a href="mailto:kinofhek@gmail.com" target="_blank">kinofhek@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">friendly ping! :)<br>
<br>
On Wed, Jun 22, 2022 at 5:44 PM Felipe Kinoshita <<a href="mailto:kinofhek@gmail.com" target="_blank">kinofhek@gmail.com</a>> wrote:<br>
><br>
> > Ah! I would suggest moving the conversion into the Controller then. As<br>
> > far as kcfg, your Config object and your Settings.qml are concerned<br>
> > they can be proper bools, it's only in the Controller that you have<br>
> > the presentation requirement that bools must be 0/1. This saves you<br>
> > the two-way conversion, in the Controller you only need to convert<br>
> > bool=>int and the rest of the app can treat them as proper bools.<br>
><br>
> Indeed, this makes much more sense.<br>
><br>
> Fixed it now!<br>
><br>
> Thanks,<br>
> Felipe<br>
><br>
> On Wed, Jun 22, 2022 at 10:10 AM Harald Sitter <<a href="mailto:sitter@kde.org" target="_blank">sitter@kde.org</a>> wrote:<br>
> ><br>
> > On Wed, Jun 22, 2022 at 12:07 AM Felipe Kinoshita <<a href="mailto:kinofhek@gmail.com" target="_blank">kinofhek@gmail.com</a>> wrote:<br>
> > ><br>
> > > > Could you elaborate why your config.kcfg uses Ints for everything when<br>
> > > > you clearly want booleans (e.g. `Config.code == 1 ? true : false`)<br>
> > ><br>
> > > The API expects ones and zeros for its params, I chose to convert them to<br>
> > > booleans to make the API call easier to write and change.<br>
> ><br>
> > Ah! I would suggest moving the conversion into the Controller then. As<br>
> > far as kcfg, your Config object and your Settings.qml are concerned<br>
> > they can be proper bools, it's only in the Controller that you have<br>
> > the presentation requirement that bools must be 0/1. This saves you<br>
> > the two-way conversion, in the Controller you only need to convert<br>
> > bool=>int and the rest of the app can treat them as proper bools.<br>
> ><br>
> > HS<br>
</blockquote></div>
</blockquote></div>