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.

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

Cleanup #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Cleanup #1

wants to merge 3 commits into from

Conversation

shobhitpuri
Copy link
Member

Lots of pending cleanup including

  • Improving the flow of Activities.
  • Bette Logging
  • Added Utilities.
  • Removed code repetition etc.

- Moved the classes to proper packages.
- Includes changes to files after applying auto-formatting for consistency throughtout the project files.
- Updated .gitignore to incluide .idea folder which is not required to be shared and is auto-generated.
- Updated the splash view to show properly when in landscape mode.
- Took out POJO classes from activity and made into own separate class.
- Refactored the changable URLS and buckets etc to their own Settings class.
- Moved some constants out of the Java files.
- Made a BaseActivity which will have all common stuff to use used by all other activities.
- Made a SharedPreferenceUtil that will contain all SharedPreference related code.
- Removed old not needed commented code.
- Added Utils
- Added Logging class to improve logs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants