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

feat: chat widget for DocsGPT #606

Merged
merged 14 commits into from
Mar 15, 2023
Merged

Conversation

tyn1998
Copy link
Member

@tyn1998 tyn1998 commented Mar 13, 2023

Brief Information

This pull request is in the type of (more info about types):

  • build
  • ci
  • docs
  • feat
  • fix
  • perf
  • refactor
  • test

Related issues (all available keywords):

Details

Working on this feature...A demo looks like this:

image

Todo:

  • check whether current repo is supported by the DocsGPT service (check this file)
  • prompt users with how to contact us for new docs support
  • handle repo switches
  • themes for light and dark
  • lock input and trigger animation when "thinking"
  • add history to requests
  • message bubbles are too narrow
  • allow disable/enable
  • issue link for requesting new docs

Next time:

Checklist

Others

@menbotics menbotics bot added the area/document Category issues or prs related to document. label Mar 13, 2023
@tyn1998 tyn1998 marked this pull request as draft March 13, 2023 14:56
@tyn1998
Copy link
Member Author

tyn1998 commented Mar 14, 2023

DocsGPT-hypercrx-demo.mov-9mb.mp4

Basicly it works :)

@tyn1998
Copy link
Member Author

tyn1998 commented Mar 14, 2023

Dark Light
image image

@tyn1998
Copy link
Member Author

tyn1998 commented Mar 14, 2023

@tyn1998 tyn1998 changed the title WIP: feat: chat widget for DocsGPT feat: chat widget for DocsGPT Mar 14, 2023
@tyn1998 tyn1998 marked this pull request as ready for review March 14, 2023 15:44
@tyn1998
Copy link
Member Author

tyn1998 commented Mar 14, 2023

Hi @lhbvvvvv @wj23027 @andyhuang18 @frank-zsy @wxharry, could you please fetch this PR and have a test?

@wj23027
Copy link
Collaborator

wj23027 commented Mar 14, 2023

I have test it , everything works well~

And I have a question. Is this emoji unclickable?
1678811886(1)

@andyhuang18
Copy link
Collaborator

It seems good~

image

@tyn1998
Copy link
Member Author

tyn1998 commented Mar 15, 2023

@andyhuang18 @wj23027 thank you for the tests!

And I have a question. Is this emoji unclickable?

Yes, I disabled emoji because I lost patience yesterday to fine tune the styles for the two themes (disabling emoji panel could save a lot of time). Maybe I should enable it and tune its style today 😆 Next time, too time consuming

image

hi @lhbvvvvv, you could test the feature with dark theme being set and upload a screenshot here :)

@l1tok
Copy link
Collaborator

l1tok commented Mar 15, 2023

The feature with dark theme works well!

image

@tyn1998
Copy link
Member Author

tyn1998 commented Mar 15, 2023

Figured a way out to change emoji picker color:

image

But the emoji picker will remain disabled until it is fine tuned for both themes. Let's leave the remaining TODOs to another PR 😆

Hi @frank-zsy, do you think it's time to merge this PR and release a new version?

@frank-zsy
Copy link
Contributor

Great, no need to fix all the minor things in this PR, let's merge this PR and release a new version!

/approve

@menbotics menbotics bot added the pull/approved If a pull is approved, it will be automatically merged label Mar 15, 2023
@menbotics menbotics bot merged commit 947b7ee into hypertrons:master Mar 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/document Category issues or prs related to document. kind/feature Category issues or prs related to feature request. pull/approved If a pull is approved, it will be automatically merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] I would like a new chat bot feature for Hypercrx
5 participants