<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jul 6, 2015 at 1:51 AM, Dirk Hohndel <span dir="ltr"><<a href="mailto:dirk@hohndel.org" target="_blank">dirk@hohndel.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Grace<br>
<span class=""><br>
On Sun, Jul 05, 2015 at 11:00:31PM +0300, Grace Karanja wrote:<br>
><br>
> This week, I managed the following:-<br>
><br>
> - Adding a preferences dialog for the user to store cloud credentials.<br>
> - Loading dives/Saving them to the cloud.<br>
> - Displaying the dive profile in QML.<br>
<br>
</span>Things looks nice, great progress. Styling is a bit off on my display (text<br>
seems to be too close to the bottom of the surrounding box) but it's quite<br>
usable already. It's just so cool when you add your credentials and it<br>
simply shows all your dives...<br></blockquote><div><br></div><div>Thanks. I am working on improving the UI issues. Once I have a dive profile in</div><div>place, I should be able to implement some swipe gestures. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
</span>That will be cool. I am really happy how close to useful this is already -<br>
and we only at about half time of the project!<br>
<span class=""><br>
> and<br>
> - Spend time on improving Android compilation. As the app has only been tested<br>
> on desktop, we will need to run it on Android for better testing.<br>
<br>
</span>Definitely. Please keep asking Anton(glance) to help you there. Worst case<br>
I can try and dig into what I can find... I don't consider fixing the<br>
cmake problem part of your GSoC tasks and don't want you to have to waste<br>
too much time on this.<br>
<span class=""><br>
> All of the work done so far is on my GitHub repository.<br>
<br>
</span>I pulled from there to play with things. I'm not quite ready to pull this<br>
into master because there seems to be too much redundant code movements.<br>
One patch moves code here, the next patch moves it there.<br>
One patch comments out code, the next patch remove the comment.<br>
Several patches have incremental changes to the same file.<br>
<br>
I know that you have used git rebase -i in the past. I'd really like to<br>
see you combine some of these commits so the series of patches becomes<br>
more structured and goes reasonably straight to the end result.<br>
<br>
>From a commit message point of view I think you are doing well. One thing<br>
that would be nice would be a "tag" in the commit title. For example if<br>
all of the commits that deal with the QML UI started with "QML UI:" and<br>
continued in lower case. Something like<br>
<br>
QML UI: implement saving dives in the cloud<br></blockquote><div><br></div><div>Thanks...I will redo the commits in this format :)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I know, this is nit picking, but the goal here is to turn you into a<br>
strong contributor :-)<br>
<br>
Thanks for your hard work.<br>
<span class="HOEnZb"><font color="#888888"><br>
/D<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">--<div>Grace K</div></div></div>
</div></div>