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

Release with latest Angular support #121

Closed
Heatmanofurioso opened this issue Jan 28, 2021 · 32 comments
Closed

Release with latest Angular support #121

Heatmanofurioso opened this issue Jan 28, 2021 · 32 comments

Comments

@Heatmanofurioso
Copy link

Hi @HaithemMosbahi. Any possible ETA on the release of the latest changes?
If there is any issue, or anything I can help with towards it, I'd gladly take up any issue

@odahcam
Copy link
Collaborator

odahcam commented Jan 29, 2021

Sorry, no ETA, Haithem is the only one who can release packages this moment.

@Heatmanofurioso
Copy link
Author

He seemed to stop being active a while after a year ago.

I hate to say this, but should we maybe consider forking and continuing the package ourselves? I've sent him an email around a month ago asking about the dependency updates before I decided to do them myself, but he didn't answer.

And I think that, with the due reference, we should consider continuing the project ourselves @odahcam

@odahcam
Copy link
Collaborator

odahcam commented Jan 31, 2021

Yeah, this is a recorrent question now. I won't be an active contributor as well, but I can in fact help maintain. @PowerKiKi is a good contributor and his opinion matters in that question.

@PowerKiKi
Copy link
Contributor

@Heatmanofurioso, we've been on the verge of forking a few months ago, but have been saved in extremis. But since then, the whole thing died down again. So I recently made up my mind to migrating away from this lib. My opinion on things didn't change since then.

@Heatmanofurioso
Copy link
Author

Heatmanofurioso commented Feb 1, 2021

I don't see myself having much time to be a main contributor, but I could easily also take part in helping maintain the library.

I think the library has some decent potential to be kept alive, and the biggest issue in forking it would probably be bringing attention to the forked version. "That, and I'd like to avoid it as much as possible, since I am a strong supporter of not trying to take ownership of something I shouldn't. But HaithemMosbahi seems to have gone inactive, and I think it's what we should do".

@odahcam Would you like to take ownership of it then? Or should I if no one else volunteers?

EDIT: I've tried contacting HaithemMosbahi one last time also, to see if he might share ownership of the repo, but if he doesn't, I feel that the opensource spirit would be for us to either drop it, or work on it

@odahcam
Copy link
Collaborator

odahcam commented Feb 1, 2021

I would take ownership if needed, no problem. But I would just maintain the repsitory and probably wouldn't implement many things.

@Heatmanofurioso
Copy link
Author

Sounds great @odahcam!
Anyway, I was able to reach HaithemMosbahi through other means "I believe I wasn't too invasive/bothersome". He said he'll try and get some time to help us out with this issue, so let's wait for a reply on his side once he has time for it :)

@PowerKiKi
Copy link
Contributor

If you can get a hold of HaithemMosbahi, I strongly suggest you ask for total control on this repository and the related npm registry (for one or both of you guys), or else fork the whole project. Anything else would be a waste of time for both parties.

@mhosman
Copy link

mhosman commented Feb 25, 2021

Yeah, we need someone else to keep updating this repo & publish new versions. @HaithemMosbahi can you please give access to other people to improve this library? That would be great! Thanks!

@rwdroge
Copy link

rwdroge commented Mar 2, 2021

fingers crossed...

@Heatmanofurioso
Copy link
Author

Well. Apparently the owner hasn't had time to reach out to us and help us out with this after that initial talk.
Do you guys think it's worth trying to reach out to him again, or should we fork this with the correct reference?

As I said.. I'm not much in favour of "taking someone's work", but in the open-source spirit, I would like to maintain this project, and also to keep on using it
@odahcam @mhosman @PowerKiKi

@PowerKiKi
Copy link
Contributor

Fork right now, with the appropriate attribution, and be done with it. The MIT license allows you do pretty much anything you want. And it's been months that several people tried to establish a working collaboration, but it never happened, because the owner did not share enough accesses. Fork, rename, make a respectful announcement around here, and move the community to something that is maintained (collaboratively).

@mhosman
Copy link

mhosman commented Mar 3, 2021

We can create a package ngx-avatars (plural) and that's it.

@Heatmanofurioso
Copy link
Author

Heatmanofurioso commented Mar 4, 2021

Anyone opposes on me making a fork on that namespace and attempting to make the package on npm?
Or anyone else wishes to take it? Also, although we do want to make reference to the old package, and give the appropriate attribution, I believe we should give it a different name, as it might be confusing to people the similarity between both, and people might mistakenly use the wrong package

@Heatmanofurioso
Copy link
Author

Also... any suggestions for the name? 😆

@mhosman
Copy link

mhosman commented Mar 4, 2021

Just put ngx-avatars and let's give access to at least two contributors to the package (I'm saying, just to speed up things). If you need help please let me know.

@Heatmanofurioso
Copy link
Author

Heatmanofurioso commented Mar 5, 2021

The repo has been created at https://github.com/Heatmanofurioso/ngx-avatars
Please give a look on the PR guys

EDIT:
Please give a look at this discussion.
Heatmanofurioso/ngx-avatars#4

@Heatmanofurioso
Copy link
Author

In case anyone is interested, the package is published and working at ngx-avatars

@mhosman
Copy link

mhosman commented Mar 9, 2021

Maybe we could put a note in the README in this repo, telling that is deprecated and the project will continue as ngx-avatars.

@Heatmanofurioso
Copy link
Author

@odahcam If you agree, could you see to it?

@HaithemMosbahi
Copy link
Owner

Hello guys,

Sorry for not being active for a long period.

I just published a new version with the changes that you merged over the last months.

Who would be interested in being added to the npm package maintainers list, that way I don't block the release of new versions when I am not available.

@Heatmanofurioso
Copy link
Author

Hi Haithem.
I'd like to be added.
If we feel like we can maintain this project, I'd much rather do that than have a clone.

I'll gladly deprecate that child project if we can keep this one up :)

@HaithemMosbahi
Copy link
Owner

Great!
Could you share with me your npm user name ? I will send you an invite

Thank you for your help

@mhosman
Copy link

mhosman commented Apr 5, 2021

@PowerKiKi Can you please also share your npm user? It could be great to have at least 2 users in addition to @HaithemMosbahi

@PowerKiKi
Copy link
Contributor

I am https://www.npmjs.com/~powerkiki. You should definitely add https://www.npmjs.com/~odahcam who has been helping here even longer than me, and https://www.npmjs.com/~heatmanofurioso who owns the new fork.

@HaithemMosbahi
Copy link
Owner

Sounds good! I will add you as maintainers.
Thanks a lot for your help and contributions

@HaithemMosbahi
Copy link
Owner

@PowerKiKi @odahcam @Heatmanofurioso I just added you as maintainers to the npm package. You should have received an invite by now.

@PowerKiKi
Copy link
Contributor

I can confirm I have access, though somehow it is not visible publicly, probably some stale cache:

image

@Heatmanofurioso
Copy link
Author

Same here. Although I'm not sure if you show up as a collaborator only once you effectively collaborate "do a publish".
But that doesn't matter.

Thank you for your time and attention @HaithemMosbahi

I'm gonna do a PR taking Angular 11.2.x in consideration here "once I have time", and once we can support the latest version, I'm going to deprecate my forked project, and point it back here

@odahcam
Copy link
Collaborator

odahcam commented Apr 7, 2021

@HaithemMosbahi is there a way you can give us permissions to change the secrets of this repository? That would allow us to configure GitHub Actions to test and auto-release the master branch!


Edit: I forgot to thank you @HaithemMosbahi for giving us that privilege, thanks!

@PowerKiKi
Copy link
Contributor

I recommend creating a GitHub org, transfer ownership of this repository to that org, and then increase access privilege (to admin) for some of us.

@Heatmanofurioso
Copy link
Author

This convo seems to be addressed on #125
Therefore, I'm closing this issue so we can approach the whole discussion there

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

No branches or pull requests

6 participants