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

Track a daily contribution goal #10

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

Conversation

KCreate
Copy link
Collaborator

@KCreate KCreate commented Jun 7, 2016

I've added the feature to track daily contribution goals.
This adds the following environment variables to the .env config file:

  • CONTRIBUTION_GOAL_DAILY_TRACKING - enable / disable
  • CONTRIBUTION_GOAL_DAILY - The daily contribution goal

If CONTRIBUTION_GOAL_DAILY_TRACKING is set to true, the app now behaves as follows:

  • If the daily contributions are 0, the emoji is set to 💔
  • If the daily contributions are lower than the daily goal, the emoji is set to 📈
  • If the daily contributions are higher or equal than the daily goal, the emoji is set to ⭐

This pull request also fixes a bug where setting CONTRIBUTION_GOAL_TRACKING to false would still show the contribution goal.

screenshot 2016-06-07 at 19 03 10

Any more thoughts on this?

@KCreate
Copy link
Collaborator Author

KCreate commented Jun 13, 2016

Are there any updates on this?

@KCreate
Copy link
Collaborator Author

KCreate commented Dec 11, 2016

Bump

@Roland0
Copy link

Roland0 commented Mar 16, 2017

hey @KCreate Thanks for this, I've got it up and running. I'm not sure @Shikkic is going to merge your PR though 🙁

@KCreate
Copy link
Collaborator Author

KCreate commented Mar 16, 2017

He probably forgot about it 😞

@KCreate
Copy link
Collaborator Author

KCreate commented Apr 13, 2017

Hey @Shikkic, if you have no objections I would love to maintain Shikkic/gh-scrape and Shikkic/gitbar . Please let me know what you think about this.

Repository owner deleted a comment from 19890843006 Feb 23, 2024
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