issue/62 - Gamification feature

KimberleyEvans-Parker commented 4 years ago

Added features:

View your stats

Currently uses hardcoded data for hours worked and 1-day record.

Productivity mode:

Can choose what level of work you want to do Currently doesn't affect anything

Points

You gain points equal to the duration of the task when you complete it (and lose them if you uncomplete it) Currently points aren't stored anywhere, so they don't persist Points example

Streak

Equal to the number of past tasks you've completed without missing one

Market Place

The user can spend their points to unlock themes.
All but the first two are dummy themes (linked to #235 ) As the points are not yet stored anywhere, they are not the same 'points' gained from completing tasks yet and therefore there's no check to make sure the points don't go negative. The unlocked themes are not stored anywhere, so they don't persist and just re-lock themselves on refresh Market place example

Issues to create/complete

As this is a big issue/PR, we were going to merge this and then create issues for the bugs we currently have so that others can also work on them

Real-time data to stats
Productivity mode affects something
Display error message when trying to purchase an item you cannot afford

Notes

closes #62

[ ] The pull request is complete according to the following criteria:
- [x] Acceptance criteria have been met
- [ ] The documentation is kept up-to-date
- [ ] Comprehensive tests (if applicable) have been generated and all pass.
- [x] The pull request describes the changes that have been made, and enough information is present in the description for any developer to understand what has changed
- [x] Commits have been squashed (or will be on merge).
- [ ] The branch name is descriptive and follows the pull request title format : {issue/bug...}/(Issue Number) - Name of issue. E.g bug/30-Fix-Project
- [x] The pull request title is of the following format : {issue/bug...}/(Issue Number) - Name of issue. E.g bug/30-Fix-Project
- [x] The description uses github syntax to link to the issue. E,g Resolves se701g2/Doto#{Number}
- [ ] At least two reviewers assigned. One of which must be the assigner of the issue.
- [ ] If there are merge conflicts, run git rebase as opposed to git merge with master.

preetpatel commented 4 years ago

oh my

Matteas-Eden commented 4 years ago

It doesn't look like there is any documentation describing the process, so is it expected that a user will learn the system purely through using it?

jordansimsmith commented 4 years ago

It doesn't look like there is any documentation describing the process, so is it expected that a user will learn the system purely through using it?

Maybe this could be a separate issue?

utri092 commented 4 years ago

I've added a lot of comments, but most are them are non blocking (just syntax / code quality things). Really great work guys, this is a really cool feature! Will be awesome to demo! 🎉

The reason I have blocked the PR, is I think there is a frontend dependency installed in the root package, which should be in the frontend package. Hit me up if this doesn't make sense.

Yeah. Jordan. We were hesitant about that one. I must yarn add in doto-frontend right?

jordansimsmith commented 4 years ago

Yeah, add it in doto-frontend and run yarn remove from the root too.

On Fri, Apr 24, 2020 at 5:27 PM utri092 notifications@github.com wrote:

I've added a lot of comments, but most are them are non blocking (just syntax / code quality things). Really great work guys, this is a really cool feature! Will be awesome to demo! 🎉

The reason I have blocked the PR, is I think there is a frontend dependency installed in the root package, which should be in the frontend package. Hit me up if this doesn't make sense.

Yeah. Jordan. We were hesitant about that one. I must yarn add in doto-frontend right?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/se701g2/Doto/pull/281#issuecomment-618809204, or unsubscribe https://github.com/notifications/unsubscribe-auth/AELBF4TPDHDJUQDHMVRUPT3ROEPMPANCNFSM4MOASUKA .

utri092 commented 4 years ago

@all-contributors please add @utri092 for code, design

allcontributors[bot] commented 4 years ago

@utri092

I've put up a pull request to add @utri092! :tada:

se701g2 / Doto