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

Update dependencies #374

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Update dependencies #374

wants to merge 8 commits into from

Conversation

darabos
Copy link

@darabos darabos commented May 19, 2023

Hi! Thanks for this library! I love the way it looks! There's even little animations. Super nice! (For the API I would have been fine with something simpler. Just messages to display in an array would be fine.)

I added it to my project. Everything was great, except the default purple didn't go well with our design. No problem, I'll just install styled-components and... npm i styled-components crashes. Probably because of my ancient dependencies. So I upgraded everything in our project to the latest version. After fixing a million issues, there's just one warning in the console: ChatBot uses the deprecated componentWillMount API. It also doesn't work anymore.

Instead of downgrading I went for updating react-simple-chatbot. I fixed issues with new dependency versions with little thought. But it's working now! The tests even pass except for one. Let me know if you want to adopt this in some way.

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

Successfully merging this pull request may close these issues.

None yet

1 participant