search

dougblack / nextbus

GT NextBus - An Android app
17 stars 6 forks source link

Feature regression, Maps? #5

Closed darkzeroman closed 11 years ago

darkzeroman commented 11 years ago

Is it possible to take out the map feature in the next version update?

I'm having the hard time debugging the maps feature. Do you think it's something worth to develop before pushing the next version?

dougblack commented 11 years ago

Sorry, but pushing without maps isn't really an option. Maps was the highest value feature I've added. Downloads really took off after Maps and I've gotten tons of great feedback on it.

Maybe @keveleigh could make sure everything checks out and that maps isn't broken?

darkzeroman commented 11 years ago

Ok,

I've figured it out.

Debugging the Map code is not something I'm able to do now, so what we will do is take the old code that is used for the Maps and leave it intact for the new version.

Downside is that there will be two classes that will be copied/pasted in, ("APIController_old.java" and "Data_old.java").

This is acceptable, there is repeated code, but we can go through it at a later date.

Close the issue if you're fine with that.