fangpenlin / avataaars

React component for avataaars
MIT License
792 stars 169 forks source link

Lots of bug fixes and program improvements #50

Open soulteary opened 3 years ago

soulteary commented 3 years ago

This project has some historical PRs that cannot be merged because of file conflicts, so I updated the code and resolved the conflict. Below is the merged code (Thank these authors for their hard work).

In addition, I upgraded the dependencies to the latest, which resolved the security issues that prompted when installing the dependencies.

And it solves a warning generated when the no-op function is executed when the server is rendering. Currently, the newer v16 version and the latest version of react are supported.

I wonder if the maintainer can merge these changes and release a new version?

Before the author merges the code, if you have a need for the above functions, you can install a temporary replacement package:

npm i @bugfix/avataaars --save
fangpenlin commented 3 years ago

Hi @soulteary,

Thanks for looking into this. Unfortunately I am very busy and didn't find time to handle those PRs. If possible, I would prefer to merge the original author's PRs to give them the credit. But I understand they were created long time ago, may not be mergeable now. I tried to merge the first two and others are all having conflicts. Do you mind help me ask the original PR authors by leaving a comment there. If they didn't respond in a week then I will review and merge your PR if it looks good to me.

soulteary commented 3 years ago

Hi @soulteary,

Thanks for looking into this. Unfortunately I am very busy and didn't find time to handle those PRs. If possible, I would prefer to merge the original author's PRs to give them the credit. But I understand they were created long time ago, may not be mergeable now. I tried to merge the first two and others are all having conflicts. Do you mind help me ask the original PR authors by leaving a comment there. If they didn't respond in a week then I will review and merge your PR if it looks good to me.

Thank you for your reply @fangpenlin . If you merge, I suggest reset my last two commits. These two commits are used to serve other projects that rely on avataaars before the PR merge.

bump ver 3a360c0
upd repo name f4955d6
soulteary commented 3 years ago

I was trying to add this repo temporarily to my package.json until it hopefully gets merged like so:

"avataaars": "soulteary/avataaars#master",

But when using yarn I get error Can't add "avataaars": invalid package version "2021.0801.0813". Is the semver invalid?

This software was not created by me, so I published this software package to @bugfix, with instructions in the original post.

Before the author merges the code, if you have a need for the above functions, you can install a temporary replacement package: npm i @bugfix/avataaars --save

@CapsAdmin

CapsAdmin commented 3 years ago

I understand, but I thought maybe your version bump could be an invalid semver.

soulteary commented 3 years ago

I understand, but I thought maybe your version bump could be an invalid semver.

Yes, this version is only a temporary version to avoid collisions with the original repo. And as a temporary solution for the other two projects.

You can read the docker section in the README, and try yourself :D

@CapsAdmin