ngx-scroll-to icon indicating copy to clipboard operation
ngx-scroll-to copied to clipboard

feat(lib): support for angular v13

Open Jonnyprof opened this issue 3 years ago • 18 comments

refactor: replace tslint with eslint refactor: change protractor to cypress build: update third party libraries

Jonnyprof avatar Jun 02 '22 10:06 Jonnyprof

@nicky-lenaers Could please take this up?

igorlino avatar Jun 08 '22 09:06 igorlino

@nicky-lenaers Could please take this up?

I will merge this PR and publish the new version ASAP

rafa-suagu avatar Jun 08 '22 18:06 rafa-suagu

Thanks @rafa-as!

HarelM avatar Jun 08 '22 19:06 HarelM

@Jonnyprof any chance to include angular v14 support?

igorlino avatar Jun 09 '22 07:06 igorlino

Thanks @rafa-as , I am not actively maintaining this package anymore

nicky-lenaers avatar Jun 09 '22 10:06 nicky-lenaers

Thanks @rafa-as , I am not actively maintaining this package anymore

Would you mind signing it over? The package is still very popular and useful to many people.

BTW: I wrote you a mail some months ago about it, but haven't received a reply yet.

andreas-aeschlimann avatar Jun 10 '22 14:06 andreas-aeschlimann

@andreas-aeschlimann Thanks for your reply. I must have overlooked your email (still can't find it though), sorry for that. I added you as a collaborator to this repo. Is this sufficient for you to continue?

nicky-lenaers avatar Jun 13 '22 13:06 nicky-lenaers

We are working to make the v14 version compatible also

rafa-suagu avatar Jun 13 '22 14:06 rafa-suagu

@andreas-aeschlimann Thanks for your reply. I must have overlooked your email (still can't find it though), sorry for that. I added you as a collaborator to this repo. Is this sufficient for you to continue?

Don't worry, it wasn't that important (logs say it was on Mar 23, 23:51).

Thank you for your quick reply. If @rafa-as wants to help, maybe you can add him as well?

The only question that remains is how we are going to publish it. If we want to continue to use the current NPM repository, you would need to give us access there as well (it seems to be simple and possible on a per-repo basis). Otherwise we would need to start a fork.

andreas-aeschlimann avatar Jun 13 '22 15:06 andreas-aeschlimann

@andreas-aeschlimann Thanks for your reply. I must have overlooked your email (still can't find it though), sorry for that. I added you as a collaborator to this repo. Is this sufficient for you to continue?

Don't worry, it wasn't that important (logs say it was on Mar 23, 23:51).

Thank you for your quick reply. If @rafa-as wants to help, maybe you can add him as well?

The only question that remains is how we are going to publish it. If we want to continue to use the current NPM repository, you would need to give us access there as well (it seems to be simple and possible on a per-repo basis). Otherwise we would need to start a fork.

@andreas-aeschlimann I have npm permissions to publish new versions ;)

rafa-suagu avatar Jun 13 '22 15:06 rafa-suagu

@andreas-aeschlimann Thanks for your reply. I must have overlooked your email (still can't find it though), sorry for that. I added you as a collaborator to this repo. Is this sufficient for you to continue?

Don't worry, it wasn't that important (logs say it was on Mar 23, 23:51). Thank you for your quick reply. If @rafa-as wants to help, maybe you can add him as well? The only question that remains is how we are going to publish it. If we want to continue to use the current NPM repository, you would need to give us access there as well (it seems to be simple and possible on a per-repo basis). Otherwise we would need to start a fork.

@andreas-aeschlimann I have npm permissions to publish new versions ;)

Ah, great. I think the most important thing is that we have a release for each Angular version (if necessary). I'll be available if you need support.

andreas-aeschlimann avatar Jun 13 '22 19:06 andreas-aeschlimann

Hey all. Can we merge and publish this to npm? Looks like the PR is ready

mparpaillon avatar Jun 26 '22 15:06 mparpaillon

Hey all. Can we merge and publish this to npm? Looks like the PR is ready

Yes please! Would greatly appreciate it @rafa-as

dxs0319 avatar Jun 27 '22 22:06 dxs0319

@rafa-as When you will merge this PR?

Alessandroinfo avatar Jul 05 '22 09:07 Alessandroinfo

@rafa-as Is there any timeline for merging this PR? It would be immensely helpful!

almoral avatar Jul 12 '22 13:07 almoral

@rafa-as, is there a specific reason why we are waiting with the update? I would do it, unfortunately I only received permissions for this repo and not for NPM. We should really publish the update, Angular 14 is already out as well.

andreas-aeschlimann avatar Jul 18 '22 12:07 andreas-aeschlimann

@andreas-aeschlimann You now have NPM access to publish versions. This should allow you and all those who are waiting for this PR to move on :)

nicky-lenaers avatar Jul 18 '22 12:07 nicky-lenaers

Whats the holdup now? :)

TheDelta avatar Aug 12 '22 15:08 TheDelta

@andreas-aeschlimann @rafa-as I think many would appreciate an answer ;)

@nicky-lenaers in case of no response, should someone else take over, including going through open issues and clean things up?

TheDelta avatar Sep 27 '22 17:09 TheDelta

@TheDelta Sure, let me know if you're interested:-)

nicky-lenaers avatar Sep 27 '22 17:09 nicky-lenaers

To @TheDelta and the others, I'm so sorry about the silence, for personal reasons I couldn't resolve this PR and the other issues. We are working to finally merge this PR, publish the version and create a new PR for the v14 version, the last one supporting non-ivy compilations.

Thanks for your patience!

rafa-suagu avatar Sep 28 '22 11:09 rafa-suagu

Thanks for the good news! Looking forward to it 🚀 ❤️

TheDelta avatar Sep 29 '22 05:09 TheDelta

To @TheDelta and the others, I'm so sorry about the silence, for personal reasons I couldn't resolve this PR and the other issues. We are working to finally merge this PR, publish the version and create a new PR for the v14 version, the last one supporting non-ivy compilations.

Thanks for your patience!

Thanks @rafa-as. Is it just a matter of merging and publishing, or is there still work to do? As I have been added to this repo just recently, I didn't want to go ahead and do it without your feedback. I would happily do it if I got the "clearance" from you guys.

andreas-aeschlimann avatar Sep 29 '22 20:09 andreas-aeschlimann

@andreas-aeschlimann Please go ahead

nicky-lenaers avatar Oct 20 '22 08:10 nicky-lenaers