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

Issue 244 - update KPI numbers on the website #250

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

Conversation

ehemp
Copy link

@ehemp ehemp commented Dec 9, 2024

Closes #244 - [UI] update numbers on the website

Created API request to GitHub for Stars and Contributor counts for DataCard.tsx

Changes

  1. Created apiDataCard.ts, configured DataCard count numbers for fetching actual data - Subscribers explicitly set, Contributors & Stars set by API
  2. Edited subtext for Contributors and Stars to be grammatically correct and removed '+'.

Type of change

  • Bug fix
  • Feature update
  • Breaking change
  • Documentation update

Flags

  1. This is my first time contributing to open source. Please message me with code formatting issues that need fixing. I would love any and all feedback as I have never worked on a project with a team.
  2. Also, my first time working with TypeScript.

Demo

DataCardChanges

How has this been tested?

  1. Checked via running web server with Docker and localhost:3000 via chrome browser.

Author Checklist

  • Code has been commented, particularly in hard-to-understand areas
  • Changes generate no new warnings
  • Vital changes have been captured in unit and/or integration tests
  • New and existing unit tests pass locally with my changes
  • Documentation has been extended, if necessary
  • Merging to main from fork:issue_244

Additional context

  1. Please message me with correct formatting. I would like to contribute to a team properly as I am actively searching for a job in this type of field and need real world experience. Any feedback and tips is greatly appreciated, thanks.

ehemp added 6 commits December 8, 2024 14:36
…count numbers for fetching actual data - Subscribers, Contributors, Stars
…count numbers for fetching actual data - Subscribers, Contributors, Stars
…count numbers for fetching actual data - Subscribers, Contributors, Stars
…count numbers for fetching actual data - Subscribers explicitly set, Contributors & Stars set by API
…count numbers for fetching actual data - Subscribers explicitly set, Contributors & Stars set by API
…count numbers for fetching actual data - Subscribers explicitly set, Contributors & Stars set by API. Added comments.
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Welcome to Django India! 🎉🇮🇳

We're so happy to see you contribute. Your efforts mean a lot to us and we're excited to have you on this journey.

Before we proceed, please take a moment to review our contribution guide. It's packed with all the information you need to make your contribution seamless.

If you're fixing an issue from the Django India issue tracker, remember to get it assigned to you before you acutally start working on it.

If you ever need help or just want to chat, the Django India discord server is always open. We're here for you!

And while you're here, don't forget to ⭐ star our repo to show your support! It helps the community grow.

Thank you for being a part of Django India. Together, let's make an amazing community! 🚀

@Yadavanurag13
Copy link
Contributor

Yadavanurag13 commented Dec 9, 2024

@ehemp Consider providing a clearer PR title for better understanding.

options: {method: "GET", headers: {'X-GitHub-Api-Version': '2022-11-28'}},
};
export const url = {
stargazers_count: "https://api.github.com/repos/djangoindia/djangoindia.org",
Copy link
Collaborator

Choose a reason for hiding this comment

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

can use env for these

@ehemp ehemp requested a review from DevilsAutumn as a code owner January 9, 2025 19:03
Copy link
Author

@ehemp ehemp left a comment

Choose a reason for hiding this comment

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

Added URLs to .env and .env.example and edited apiDataCard.ts to reflect the changes.

@ehemp ehemp changed the title Issue 244 Issue 244 - update KPI numbers on the website Jan 9, 2025
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.

[UI] update numbers on the website
3 participants