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

Add org links and today's contributions #2

Closed
wants to merge 2 commits into from
Closed

Add org links and today's contributions #2

wants to merge 2 commits into from

Conversation

aashutoshrathi
Copy link
Collaborator

@aashutoshrathi aashutoshrathi commented Feb 24, 2019

Fixes #1 #3

PS: Contribution date will get fetched for the new day around 1:30, due to Timezone issues.

Screenshot:
image

@aashutoshrathi
Copy link
Collaborator Author

pls check @mohitkyadav

message.channel.send(enhanceChat.embedCard(response.data));
ghUser = response.data;
}).catch(err => console.log(err));
await axios.get(`https://github-contributions-api.herokuapp.com/${username}/count`).then(res => {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only check contributions if type is user.
Getting 503 for org. i.e. !gcard google

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On it.

@mohitkyadav
Copy link
Owner

Mr. @aashutoshrathi 🙄

@aashutoshrathi
Copy link
Collaborator Author

Mr. @mohitkyadav Kherli
I'll do it as soon as my health allows.

@mohitkyadav
Copy link
Owner

@aashutoshrathi working on it?

@anshumanv
Copy link

anshumanv commented Mar 22, 2020

Kherli Mukul and Nimbahera Aashutosh, karao be isko merge. 🍀

@aashutoshrathi
Copy link
Collaborator Author

Sirf type ka check bacha h na @mohitkyadav ?

@mohitkyadav
Copy link
Owner

Dunno @aashutoshrathi will let you know when I'm on my pc.

@aashutoshrathi
Copy link
Collaborator Author

Cool. Let me know and I'll update the PR ASAP.

@anshumanv
Copy link

👍 ⚡️

@aashutoshrathi
Copy link
Collaborator Author

Ahh.. I needa fork again and create a new PR maybe.

@anshumanv
Copy link

😕

@aashutoshrathi
Copy link
Collaborator Author

No worries. @mohitkyadav Kherli and @anshumanv Japani.
I will be doing anything needed ASAP.
Just keep your Approve and Merge sticks ready. 😄

@anshumanv
Copy link

Okay @aashutoshrathi Nimbahera 👍

@aashutoshrathi
Copy link
Collaborator Author

#6

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.

GitHub card
3 participants