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

Feature/solo chat #8

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

Feature/solo chat #8

wants to merge 8 commits into from

Conversation

mooo-ooo
Copy link

@mooo-ooo mooo-ooo commented Oct 31, 2019

Hi Johan
Here is my pull request

Please take a look

Due this application is quite simple, and there are not shared state, therefore i decided to not use Redux, instead of that, i use React hook and es6+ features to reduce life cycle steps, and keep the code much simpler

Below is a screenshot
that i did type '/starwars r2' to fetch character

alt text

and please let me know if you face with any installation problem, i may host this test to cloud

Thanks !!!

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.

2 participants