TeamCritterFinder / CritterFinder

Critter Finder CodePath Project
0 stars 1 forks source link

Sprint 1: Critter Finder #41

Closed smacgregor closed 8 years ago

smacgregor commented 8 years ago

Our first sprint is complete, please review. /cc @codepathreview @codepath

@scottrichards @cbaja

Another fun project! We enjoyed getting to explore the PetFinder API and learning to work together.

Interesting Notes

We made really good progress getting the general flows and UI screens in place. We didn't focus much on making the UI pretty.

The material design library was invaluable when it came to building out search filters. We made heavy use of that library. Very slick.

Open Questions

We're looking forward to thinking about animations and how we'll implement the "Tinder" wipe mechanic for cycling through a list of pets.

Getting location from the GPS working with genymotion is an adventure. Thank you Nate for sharing the cliff notes for converting a latitude and longitude into an address. That came in handy for us and we learned how to set up an Async Task in order to do it!

codepathreview commented 8 years ago

Group Sprint 1 Review

:+1: Nice work creating wire-frames and creating stories for sprint 1. Great to see the basic screens and flow coming along well. Few notes to mention:

Good to see the basic flow coming along well. Please take notes of these points and improve upon the same in the next sprint. Look forward to the rest of the functionality and improvement on the core flows by the end of the Sprint 2 so you can have enough time to add UI polish and advanced nice-to-have features later on. It is those little extra things you do that make everything feel more polished and desirable to use.

smacgregor commented 8 years ago

Thanks for the feedback @codepathreview @codepath.

Just an aside - https://github.com/TeamCritterFinder/CritterFinder/graphs/contributors should not be used to track project contributions. That github graph only tracks commits directly to master which is discouraged. It ignores all merges made through pull requests.

We use pull requests for our commits and none of those changes will appear in that graph.

We also have been grouping issues into Sprint milestones which you can see here https://github.com/TeamCritterFinder/CritterFinder/milestones. Please let us know if you have suggestions on how we could be using milestones more effectively.

We're looking forward to wrapping up the functionality in the next sprint so we can start polishing the app!

Thanks again.

-Team Critter Finder