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

Es4 discovery library - Revamp #182

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

mickey4653
Copy link

Description

Revamped Dicovery library Feature

Ticket:

  • created a component for discovery library feature
  • integrated the discovery library feature
  • created a discovery library window
  • added images necessary for discovery library
  • integrated discovery library via the main layout to follow code pattern in the project
  • created custom prompts in the constants folder

How Has This Been Tested?

	- created a component for discovery library feature
	- integrated the discovery library feature
	- created a discovery library window
	- added images necessary for discovery library
	- integrated discovery library via the main layout to follow code pattern in the project
	- created custom prompts in the constants folder
	- adjusted styling to make discovery library feature fit well along side chat interface
@mickey4653 mickey4653 changed the title Es4 discovery library Es4 discovery library - Revamp Nov 25, 2024
Copy link
Contributor

@bkb-Git bkb-Git left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks mostly good, just need to avoid pushing code with comments in them and also let's try and make the code readable by adding whitespacing where necessary.

.firebaserc Outdated Show resolved Hide resolved
frontend/constants/prompts.js Outdated Show resolved Hide resolved
frontend/layouts/MainAppLayout/ MainAppLayout.jsx Outdated Show resolved Hide resolved
frontend/layouts/MainAppLayout/NavBar/Navbar.jsx Outdated Show resolved Hide resolved
frontend/pages/discovery/index.jsx Outdated Show resolved Hide resolved
frontend/templates/Chat/DiscoveryLibraryWindow/styles.js Outdated Show resolved Hide resolved
	- change project name to default
	- modifed constants for prompts to personas
	- added route_ids file under constants dir
	- removed comments
	- created database collection for personas
	- added thunks and slices for personas and made sure its been stored in redux
	- removed comments
	- added new lines in specified areas
	- refractored code in discovery library window
	- eslint fix
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