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

FIX #641: GitHub Calender working properly #658

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

AdityaMayukhSom
Copy link

@AdityaMayukhSom AdityaMayukhSom commented Jan 6, 2024

This pull request is a fix to the issues #641.

The old github-calender package hasn't been updated in the last 2 years and it lacks proper documentation too. Hence I submit a pull request with minimal changes introducing a react based GitHub calendar library named react-github-calender. This library is well documented (documentation) and if updated regularly.

Current new tab with light theme looks like -

image

Introducing this library reduces -

  1. The manual CSS modifications required in case of the previous library.
  2. Unnecessary module declaration required with the old library.

Note: This pull request also fixes a TypeError in storage.ts file for a setTimeout function.

- added react-github-calender
- removed github-calender
- removed old manual import of css files for github-calender
- added GitHubCalender component from react-github-calender
  and setup username prop
- react-github-calender uses it's own css so removing manual css.
- it is written in typescript, so removing unused module declaration.
@AdityaMayukhSom AdityaMayukhSom changed the title FIX #641: GitHub Calender working properly FIX #641, #169: GitHub Calender working properly Jan 6, 2024
@AdityaMayukhSom AdityaMayukhSom changed the title FIX #641, #169: GitHub Calender working properly FIX #641: GitHub Calender working properly Jan 6, 2024
@AdityaMayukhSom
Copy link
Author

@joelshepherd Could you please give any updates regarding the bug with GitHub calendar?

@EpsilonKu
Copy link

Yeah, please.

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.

None yet

2 participants