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

Draft: Add daily goals funcionality #331

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

HighPriest
Copy link

@HighPriest HighPriest commented Feb 27, 2024

Hello Adrian.

As an exercise in Android Development, I have decided to attempt extending your application with a "Daily goal" functionality.

TODO:

  • Allow enabling Daily goal in Settings
  • Allow setting Daily goal in Labels Editor
  • Display labels with incomplete Daily goal in main application view
    • ??Use a narrow RecyclerView on the side of the clock??
    • To be done, by querying DB for all sessions of the day and subtracting their length from the goal
  • Display Daily goal target in Statistics, as a line in time history. To view how we were doing in completing daily goals over time.
  • Add some form of alerts to motivate the user to change the current task.

It is my first time contributing to a foreign Android project, so any guidance as to what I might be doing incorrectly, is greatly appreciated. Especially because the project is so old, extensive and polished.

Example design of Main Activity Preview
Screenshot_20240227-233731_Productivity

WIP Issues:
- After setting the time for a label, opening the dialog keeps the old time value. Only updates when the view is closed & re-opened. Probably caused by how the dialog is created (a lambda), causing the default value to be set only on creation of the view.
- The set time doesn't show up anywhere
@HighPriest HighPriest closed this Feb 27, 2024
@HighPriest HighPriest changed the title Add functionality Draft: Add daily goals funcionality Feb 27, 2024
@HighPriest HighPriest reopened this Feb 27, 2024
@HighPriest HighPriest marked this pull request as draft February 27, 2024 22:22
@adrcotfas
Copy link
Owner

adrcotfas commented Feb 28, 2024

Thank you for your contribution but please keep in mind that I'm working on a new version of the app(from scratch) and I won't be able to use your work.

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