Skip to content
This repository has been archived by the owner on Aug 7, 2024. It is now read-only.

fix: add kingwisdom profile #745

Merged
merged 7 commits into from
Nov 29, 2021

Conversation

kingwisdom
Copy link
Contributor

@kingwisdom kingwisdom commented Nov 29, 2021

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Copy link
Contributor

@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.

It's great having you contribute to this project

Welcome to the community 🤓

If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord chat and our GitHub Organisation - we help and encourage each other to contribute to open source little and often 🤓 . Any questions let us know.

Copy link
Contributor

@gptkrsh gptkrsh left a comment

Choose a reason for hiding this comment

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

Can you reset the package-lock files and format the files to

{
  "name": "Adeoye Temitayo",
  "bio": "Software Developer || Open Source Enthusiast.",
  "avatar": "https://avatars.githubusercontent.com/u/32174632?v=4",
  "links": [
    {
      "name": "Follow me on GitHub",
      "url": "https://github.com/kingwisdom",
      "icon": "github"
    },
    {
      "name": "Follow me on Twitter",
      "url": "https://twitter.com/adeoyeTemitayo9",
      "icon": "twitter"
    }
  ]
}

@Panquesito7 Panquesito7 added the ✍ chore: profile This label will be added on PR with request of Data json addition label Nov 29, 2021
@Panquesito7 Panquesito7 changed the title adding kingwisdom profile fix: add kingwisdom profile Nov 29, 2021
Copy link
Member

@Panquesito7 Panquesito7 left a comment

Choose a reason for hiding this comment

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

Please see the comments above.

{
"name": "Adeoye Temitayo",
"bio": "Software Developer || Open Source Enthusiast.",
"avatar": "https://avatars.githubusercontent.com/u/32174632?v=4",
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"avatar": "https://avatars.githubusercontent.com/u/32174632?v=4",
"avatar": "https://github.com/kingwisdom.png",

public/data/kingwisdom.json Outdated Show resolved Hide resolved
Copy link
Member

@Panquesito7 Panquesito7 left a comment

Choose a reason for hiding this comment

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

LGTM 🚀 Thank you for your contribution! 😄👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
✍ chore: profile This label will be added on PR with request of Data json addition
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants