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

Added leetcode to socials and tried to implement github username changing #86

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

yajatkaul
Copy link

@yajatkaul yajatkaul commented Jan 9, 2025

Changes i made for leetcode

  • new file: src/assets/icons/social/leetcode/default.svg
  • modified: src/resources/social_icons.ts

Changes i made for Github username changing ability

  • modified: src/features/stats/panel/views/layout/fields.ts

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Enhancement
  • Documentation Update

What I did

  1. I added leetcode icon to socials tab
  2. I added a text field in stats named Github Username since it was hard to find a way to change the username once set
    I was unable to do the github username thing since the code was very deep

	new file:   src/assets/icons/social/leetcode/default.svg
    modified:   src/resources/social_icons.ts

Trying to add the ability to change the github username in the stats layout tab
	modified:   src/features/stats/panel/views/layout/fields.ts
Copy link

vercel bot commented Jan 9, 2025

@yajatkaul is attempting to deploy a commit to the Mauro de Souza's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Owner

Choose a reason for hiding this comment

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

I didn't understand why you changed the yarn.lock. Please remove this file from PR.

Copy link
Owner

Choose a reason for hiding this comment

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

The package management used in this project is yarn. Please remove this file from PR.

Copy link
Owner

Choose a reason for hiding this comment

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

A panel needs to be created to change the user config, and there should be an input to update the GitHub username. I understood the problem related to the GitHub username, but this is not the right place to edit it.

These changes can be removed.

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