Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Decongest MapActivity #51

Open
1 of 2 tasks
jasonrogena opened this issue Jan 5, 2018 · 0 comments · May be fixed by #98
Open
1 of 2 tasks

Decongest MapActivity #51

jasonrogena opened this issue Jan 5, 2018 · 0 comments · May be fixed by #98
Assignees
@ekigamba
Labels
Has-PR
Milestone
Week 6 - 7

Comments

@jasonrogena
Copy link
Contributor

jasonrogena commented Jan 5, 2018 •
edited by ekigamba
Loading

As a way of decongesting the MapActivity class, to make it easier to write tests for the code and in anticipation of code reuse, move the logic for the following features out of this class:

  • Location updates
  • The code in the sortData method
@jasonrogena jasonrogena added this to the Week 6 - 7 milestone Feb 2, 2018
@ekigamba ekigamba added the Has-PR label Feb 6, 2018
@ekigamba ekigamba linked a pull request Feb 6, 2018 that will close this issue
Open
ekigamba pushed a commit that referenced this issue Apr 25, 2019
@eotin
Merge pull request #51 from eotin/Develop
b882843 
Add Check storage permissions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ekigamba ekigamba

Labels
Has-PR
Projects
None yet
Milestone
Week 6 - 7
Development

Successfully merging a pull request may close this issue.

Decongest MapActivity onaio/kujaku
2 participants
@jasonrogena @ekigamba