anitab-org.github.io icon indicating copy to clipboard operation
anitab-org.github.io copied to clipboard

Published 20 hours ago • verified publisher icon anitab-org

Feat: developed contribution section in project page

Open Bucky25 opened this issue 3 years ago • 18 comments

Description

Used GitHub API to get commit data and made a contribution heat map for last 30 days patch #80

Fixes #80

Type of Change:

Delete irrelevant options.

  • Code

Code/Quality Assurance Only

  • Bug fix (non-breaking change which fixes an issue)

  • New feature (non-breaking change which adds functionality pre-approved by mentors)

How Has This Been Tested?

I have tested the implementation and responsiveness. here is the deployed link. https://bucky25.github.io/anitab-org.github.io/

Checklist:

  • [x] My PR follows the style guidelines of this project
  • [x] I have performed a self-review of my own code or materials
  • [x] I have attached link of deployed site.
  • [x] Any dependent changes have been merged

Code/Quality Assurance Only

  • [x] My changes generate no new warnings

Bucky25 avatar Dec 06 '20 17:12 Bucky25

@gaurivn can u test this PR??

nandini45 avatar Dec 12 '20 17:12 nandini45

is this the same PR as https://github.com/anitab-org/anitab-org.github.io/pull/171 ?

annabauza avatar Dec 23 '20 12:12 annabauza

@Bucky25 Please squash your commits into one commit !!

Vishal-raj-1 avatar Jan 29 '21 17:01 Vishal-raj-1

@Vishal-raj-1 we squash when merge. No need to squash.

annabauza avatar Jan 31 '21 14:01 annabauza

@annabauza that's great !!

Vishal-raj-1 avatar Jan 31 '21 14:01 Vishal-raj-1

@Vishal-raj-1 we squash when merge. No need to squash.

@annabauza thanks

Bucky25 avatar Feb 01 '21 18:02 Bucky25

@Bucky25 any pending work needs to be done in the pr??

nandini45 avatar Feb 16 '21 21:02 nandini45

@Bucky25 any pending work needs to be done in the pr??

@nandini45 i think its done i have done all the changes required ?

Bucky25 avatar Feb 18 '21 11:02 Bucky25

@Bucky25 yes i will ping Anna again for her review sorry for the delay. you can look for other issues if you want thanks

nandini45 avatar Feb 18 '21 13:02 nandini45

@Bucky25 It contains conflicting files. Resolve it !!

Vishal-raj-1 avatar Feb 19 '21 06:02 Vishal-raj-1

@annabauza please remove these files: src/assets/events_and_highlights/calendar.png src/assets/events_and_highlights/location.png src/assets/events_and_highlights/time.png as they are not correct png for icons then merging is all fine

Bucky25 avatar Feb 19 '21 09:02 Bucky25

@annabauza can you please provide your feedback ?

nandini45 avatar Feb 27 '21 20:02 nandini45

@Bucky25 any updates about solving the conflicts?

keshakaneria avatar Mar 08 '21 05:03 keshakaneria

@Bucky25 any updates about solving the conflicts?

i already mentioned that these files are existing in the repo and are different from pr because new one are that png files .

Bucky25 avatar May 05 '21 20:05 Bucky25

@annabauza can you review the PR please?

nandini45 avatar May 06 '21 18:05 nandini45

@isabelcosta Can you review the PR ?

nandini45 avatar Sep 29 '21 04:09 nandini45

@isabelcosta Can you review the PR ?

I can try but my frontend skills are a bit rusty, could you ask Zulip to see if anyone wants to help. @nandini45

isabelcosta avatar Oct 04 '21 22:10 isabelcosta

@Bucky25 are you still interested in finishing this issue?

isabelcosta avatar Oct 17 '21 20:10 isabelcosta