-
Notifications
You must be signed in to change notification settings - Fork 148
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
Modified link so multiple react components can instantiate different … #28
Conversation
@pbernasconi Any update? |
I went ahead and pulled the latest into my fork and pull request. I'm not too familiar with this process so let me know if I need to redo the pull request before merging. |
@jlaramie I'm not sure why it is in the state. Can't it be just |
@pawelmaczka State isn't necessary and could be just |
What would it take to get this merged in? It would be super useful for what I'm working on, and I'm sure for many others as well. |
# Conflicts: # package.json # src/PlaidLink.js
@pawelmaczka I modified this to not use this.setState to hold the linkHandler. I also fixed the package.json referencing lib/ instead of dist/ |
Sorry that I have been unresponsive for this repo! In the next few weeks, I'm planning to move this repo over to the official Plaid organization, and will be addressing this, and the other open issues then! Stay tuned for an update soon. |
Hi @pbernasconi! Thanks for your work on this. Are there any updates on moving this to the Plaid org? |
Would be awesome if this got merged soon! @pbernasconi |
@pbernasconi @jeffcoh23 This should be all good to go for a merge. |
@jlaramie @pbernasconi any update? |
I wish :) |
…plaid links