Code review comment for lp:~rexbron/astrid/num_keyboard_fix

Revision history for this message
Tim Su (tim-todoroo) wrote :

Awesome. Last thing: you should mark this branch as "merged" instead of
"development", since you're done with it.

Did you know that if you have an Android phone, you can do development
on your phone? The only caveat is if you already have Astrid installed
from the market, you'll have to uninstall, which means you may want to
back up your data (i.e. from /data/data/com.timsu.astrid). Let me know
if you want more info about this.

Thanks again!

Andrew Hunter wrote:
>>> Also suggesting to merge this into the 2.8.7 bugfix release.
>>>
>>> This is the first piece of Android software I've hacked on. I was able to
>>>
>> test
>>
>>> it in an emulator but don't have spare hardware to do an on device test.
>>> Hopefully I haven't made any mistakes.
>>>
>> Looks good. I'm not going to incorporate the manifest and default.properties
>> file, but otherwise, I'm going to go ahead with the merge. Thanks so much!
>>
>
> Cool, I updated the project to get the build.xml file to try and build astrid from the command line but ended up using eclipse instead.
>

« Back to merge proposal