Task 3 updates [New components from chat interface, Discovery Library UI changes from figma update] #135
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
minor style changes
- padding at the top of chat ChatHistory - made button highlight extend to timestamp
- changed chat history button - gave outline to default prompts & corrected behavior - prototpye of new chat button
- merged quick button actions - merged discovery library - slight style changes to discovery library
… into Task-3-Updates
- default prompts are uniform size - default prompt text hovers purple - minor code style changes
- made custom prompts to start a new conversation
… into Task-3-Updates
- adjusted prompt changes
- created a render function for custom prompt - added custom prompt display message on chat interface - fixed eslint errors - fixed quick action buttons to send action to chat input
- Merged all changes - All Task completed for Epic 5: Kai Chat
bkb-Git
requested changes
Sep 2, 2024
frontend/templates/Chat/Chat.jsx
Outdated
There was a problem hiding this comment.
Let's clean up this component by abstracting each part of it into their own respective components and importing them here. This size of this component is too large
frontend/templates/Chat/DiscoveryLibraryUI/DiscoveryLibraryUI.jsx
Outdated
Show resolved
Hide resolved
frontend/templates/Chat/DiscoveryLibraryUI/DiscoveryLibraryUI.jsx
Outdated
Show resolved
Hide resolved
- removed project name - undo project target name - removed comments from firebase hosting - moved fetchChathistory function into service folder - removed react import statement - made categorized prompts into constants folder - removed date-fns package from frontend/packge.json - made use of the moment library - removed some specified files defined in .gitignore file
- made some PR review changes - adjusted the discovery library to match latest figma file
- Resolved Merged Conflicts
mickey4653
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Ticket:
Type of change
Chat History:
Default Prompts:
Quick Actions:
Discovery Library UI:
How Has This Been Tested?
Here's link to the video:
https://jam.dev/c/8c9466a6-7df6-4043-a05d-412c05a04b13