search

ppatibandla / todolist

Todo list app for android boot camp
0 stars 0 forks source link

[Android Bootcamp] Review my to do list app #1

Open ppatibandla opened 10 years ago

ppatibandla commented 10 years ago

CC : @nesquena ; @thecodepath I implemented all the required features (add, remove, edit) in the to do list app. Please review it. README file has gif showing all the three features.

I am actively working on adding other optional features, I will update here when optional feature is done.

ppatibandla commented 10 years ago

CC: @nesquena ; @thecodepath I modified to do list app to use SQLite to persist data instead of text file. Please review it.

nesquena commented 10 years ago

Pavan,

Looks great! This was intended in part to give you an introduction to the general rhythm of this course. The course is entirely project based with an app being assigned each week and then due the following week. Each project builds on the last to help each engineer to learn all practical Android development and best practices as quickly as possible. We also do a code review for each submitted project.

Good to see you already were able to add sqlite persistence!

The next step is to continue working on extensions to your todo app and to schedule a short 5-10 minute phone conversation here. Navigate to August 15th and onward and choose a 15-minute slot. Let us know if none of those times work.

Once you select a slot, can you edit it with the best number to reach you at? Look forward to chatting soon!

ppatibandla commented 10 years ago

@nesquena Thanks for feedback. I added support for due date using date picker and custom adapter. Please review it.

I already picked slot for phone conversation. Looking forward to talking with you.

nesquena commented 10 years ago

Great, chat with you soon.

ppatibandla commented 10 years ago

@nesquena I added support for assigning priority to task. Also modified adapter to change background color of the item based on priority. Please review it.

I used Auto Complete Text View for priority as I want to change it to accept generic label and let user assign color to label. I am still working on this.

nesquena commented 10 years ago

This look great, glad you were able to complete these extras. I will be in touch tomorrow with further details.

ppatibandla commented 10 years ago

@nesquena Thanks for the feedback. Updated Todo app to accept custom labels. Modified code to assign random color to each label and persist the mapping. Please review it.