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

make notes and remarks #14

Closed
TomVarga opened this issue Aug 10, 2018 · 6 comments
Closed

make notes and remarks #14

TomVarga opened this issue Aug 10, 2018 · 6 comments
Milestone
First release

Comments

@TomVarga
Copy link
Owner

  • describe what was done and why (patterns, libraries used)
  • describe what was NOT done and why (what was out of scope, where corners were cut)
  • describe future plans and explandibility
@TomVarga TomVarga modified the milestones: Second r, First release Aug 10, 2018
@TomVarga
Copy link
Owner Author

databindings were not used because in my experience they kill development build times and at this point investigating if those issues have since been resolved was out of scope of this project

@TomVarga
Copy link
Owner Author

Generating DAOs would have been nice to have for example if the the API used had swagger definitions.

@TomVarga
Copy link
Owner Author

On the weather screen the bottom list of foreast could be clickable to view that day's detail data above the list.

@TomVarga
Copy link
Owner Author

Full localization support is not in scope for the first release.

@TomVarga
Copy link
Owner Author

TomVarga commented Aug 12, 2018 •
edited
Loading

Proper landscape view is not in the scope for the first release. (there is some)

@TomVarga
Copy link
Owner Author

Min SDK was set to 21 to avoid having to deal with multidex support.

TomVarga added a commit that referenced this issue Aug 14, 2018
@TomVarga
#14: make notes and remarks
2fdfb1d 
* add notes and remarks
* try and fix icons
* remove unused dependency
TomVarga pushed a commit that referenced this issue Aug 14, 2018
@calebzor
#14: make notes and remarks (#27)
2863fe6 
* add notes and remarks
* try and fix icons
* remove unused dependency
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
First release
Development

No branches or pull requests
1 participant
@TomVarga