react-intercom icon indicating copy to clipboard operation
react-intercom copied to clipboard

2.0 Refresh

Open nhagen opened this issue 4 years ago • 17 comments

Hey Folks,

This package hasn't gotten a ton of attention from me lately, but a number of issues have come up that needed to be addressed. I was able to find some time today and migrated the build system to rollup, and the component itsself to a functional component using hooks.

This means there were some significant API changes, and thus a 2.0 branch/tag, and a new 2.0 alpha release on NPM.

The biggest changes are (a) that now both exports of this package are now named exports, and (b) the package is now more strict about the props that are passed in.

There are also improvements to the TS types, and probably some other things. Happy to document anything that I've forgotten to mention here. Please leave a comment or submit a PR if you find anything worth documenting.

You can read documentation on the alpha release by looking at the 2.0 branch or 2.0.0-alpha.1 release.

You can try out the latest 2.0 release by installing react-intercom@next. This should be more or less ready to be released, but I want to get some eyes on it to confirm there aren't any issues.

Once I feel confident there won't be any issues, I'll merge this into master and push an official v2.0.0 release.

nhagen avatar Sep 01 '19 04:09 nhagen

@nhagen you can open issue here - https://github.com/suhlob/react-intercom-widget/issues

nsuglob avatar Sep 02 '19 07:09 nsuglob

Got halfway through rewriting to hooks myself before I saw this issue. Maybe there is something you can use at https://github.com/BramKaashoek/react-intercom.

BramKaashoek avatar Sep 11 '19 14:09 BramKaashoek

IntercomAPI secondary arguments don't get passed through correctly. I've opened a pull request: https://github.com/nhagen/react-intercom/pull/77

nikparo avatar Oct 28 '19 08:10 nikparo

Type mismatch on hide_default_launcher. I have opened a pull request: https://github.com/nhagen/react-intercom/pull/76

andrewjrhill avatar Oct 28 '19 08:10 andrewjrhill

@nhagen any chance we can these PRs merged and a new 2.0 release?

bluepeter avatar Oct 31 '19 07:10 bluepeter

Ping @nhagen!

Thanks for the great work! Perhaps you should add to current master README.md about this issue/pre-release?

I've stumbled upon this by mere luck. I saw last commit on master is from Oct 25th 2018 and was about to move on and skip using this package...

vfonic avatar Nov 25 '19 10:11 vfonic

Hey @nhagen , great work, do you have any plans of releasing the 2.0 version and if you do when?

BAPopov avatar Dec 09 '19 15:12 BAPopov

Would be nice if we could get @fvoordeckers PR released before the 2.0 goes out so we can get back our consoles. https://github.com/nhagen/react-intercom/pull/84

andrewjrhill avatar Dec 20 '19 14:12 andrewjrhill

Any updates on 2.0?

danvernon avatar Jan 20 '20 10:01 danvernon

At this point I feel like it is no longer in @nhagen's interest to continue driving this product forward. There are several open pull requests, and I believe it would be in everyone's interest if either a new, definite fork of the project is made, that @nhagen gives a different maintainer access to merge contributions into the repo, or that the ownership is transferred.

I have no problems understanding that people are busy, but maybe he/she/x should then pass the baton to someone else who has time to maintain/merge contributions.

ChristianJacobsen avatar Feb 26 '20 12:02 ChristianJacobsen

Two thumbs up to that.

I sent him an email...

bluepeter avatar Feb 26 '20 16:02 bluepeter

Any updates on release 2.0?

Morsmetus avatar Mar 09 '20 13:03 Morsmetus

Anyone want to fork? No reply from the maintainer...

bluepeter avatar Mar 27 '20 06:03 bluepeter

I'm keen to announce that I've released a React Intercom integration powered by hooks:
https://github.com/devrnt/react-use-intercom

Features

  • Hooks
  • Written in TypeScript
  • Documented, self explaining methods

Motivation I've found it not very developer friendly to integrate Intercom in a React application.

Note I don't want to spam this issue with advertisement, that's why I'll delete this message in the near future.

devrnt avatar May 17 '20 15:05 devrnt

It would be great if NPM and GitHub flagged projects as "dormant" if outstanding issues see no feedback from contributors/maintainers after a few weeks.

The fact that this repo is still seeing over 40,000 downloads a week with no active maintainers is pretty alarming.

andrewjrhill avatar Jun 24 '20 11:06 andrewjrhill

@andrewjrhill I fully agree with you. That's one of the flaws of open-source software. Other two big issues that I see are:

  1. Funding
  2. Taking over inactive projects

That being said, your comment written here (just like mine), provides no benefit to this repo nor to the idea you expressed in it. Perhaps a better channel to deliver your message would be by writing to GitHub employees on twitter and I believe I saw somewhere that they have some GitHub repo for suggestions (or I'm making stuff up now 🤔).

vfonic avatar Jun 24 '20 12:06 vfonic

@devrnt Thanks Jonas this is great! Please don't delete the post as it was the only way I found out about it. I think it would be great if you could contact Intercom and have your package listed instead of this one (on this page: https://www.intercom.com/help/en/articles/170-integrate-intercom-in-a-single-page-app ).

goamn avatar Apr 22 '21 14:04 goamn