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.

Sign up for GitHub

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

Jump to bottom

LLM policies UI schema #3352

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

LLM policies UI schema #3352

wants to merge 205 commits into from

Conversation

pi1814
Copy link

@pi1814 pi1814 commented Nov 21, 2024 •
edited
Loading

What does this PR do?

Adds UI Schema for LLM policies

Type of change

  • New feature (non-breaking change which adds functionality)

How should this be tested?

  • Existing unit tests

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code and corrected any misspellings
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

niwsa added 30 commits July 15, 2024 11:49
@niwsa
Tweak shared components for chat UI
cfd2d10
@niwsa
Add LLM packages
11277f6
@niwsa
Merge branch 'main' into chatui
901ca3e
@niwsa
ChatController WIP
abb223e
@niwsa
createConversation, createChat and getConversationById
ad918cf
@niwsa
Add index for conversation
e4055b9
@niwsa
Remove LLMChat member
127d2fd
@niwsa
put doesn't return anything, manually return the created object
580765f
@niwsa
TeamId will be optional and in case of admin portal just use userId f…
646a33d 
…or index
@niwsa
Remove getAllConversations as retrieval will be via an index
89bb54c
@niwsa
Merge branch 'main' into chatui
ed2033a
@niwsa
Add zod validation
cd82e28
@niwsa
LLM handler and env additions
17cda0a
@niwsa
Add terminus/llm opts
e29b3dd
@niwsa
Switch teamId with tenant, Add methods for managing LLM configs
07baf67
@niwsa
Expose chatController from npm
d84e762
@niwsa
apiKey and baseURL should be optional
d149007
@niwsa
Return the config object after save
2a35a58
@niwsa
API route for fetching and saving LLM configs
0a27ce0
@niwsa
deleteLLMConfig implementation
34ecf15
@niwsa
Fixes for deleteLLMConfig
f6020b7
@niwsa
Add updateLLMConfig method
8e0aad1
@niwsa
API route for updating/deleting LLM configs
e2c7278
@niwsa
Add internals API route
9b2c528
@niwsa
Merge branch 'main' into chatui
3e1eef2
@niwsa
Move api under [tenant]
1ff0647
@niwsa
Merge branch 'main' of ssh://github.com/boxyhq/jackson
5d42b11
@niwsa
Merge branch 'main' into chatui
9cbef62
@niwsa
Minor formatting for env
74da24f
@niwsa
Move under [tenant] slug
82da327
@deepakprabhakara deepakprabhakara changed the base branch from llm-policy to chatui November 26, 2024 14:40
deepakprabhakara
deepakprabhakara reviewed Nov 29, 2024
View reviewed changes
package.json Outdated Show resolved Hide resolved
deepakprabhakara
deepakprabhakara reviewed Nov 29, 2024
View reviewed changes
package.json Outdated Show resolved Hide resolved
deepakprabhakara
deepakprabhakara reviewed Nov 29, 2024
View reviewed changes
package.json Outdated Show resolved Hide resolved
@deepakprabhakara
Copy link
Member

@pi1814 should remove heroicons from package.json

parth inamdar added 4 commits December 2, 2024 09:08
Added package json changes
2501a78
Changed from react-simple-code-editor to monaco-editor
e726558 
- better customization options
- in editor scroller possible
Added changes:
a67de8c 
- Changed code editor from monaco to react-ace for smaller bundle size
- Added optimization for the backend code
- Added validations for the Entity and Policy selection
Reverting to monaco editor and streamlining the PolicyForms
3e368c5
deepakprabhakara
deepakprabhakara reviewed Dec 3, 2024
View reviewed changes
package.json Outdated Show resolved Hide resolved
@deepakprabhakara deepakprabhakara changed the base branch from chatui to main December 3, 2024 12:01
@deepakprabhakara deepakprabhakara mentioned this pull request Dec 3, 2024
Closed
13 tasks
@deepakprabhakara
Merge branch 'main' into llm-policies-ui-schema
0917296 
# Conflicts:
#	internal-ui/package-lock.json
#	npm/package-lock.json
#	package-lock.json
#	package.json
@deepakprabhakara
Merge branch 'main' into llm-policies-ui-schema
7ebabb9 
# Conflicts:
#	internal-ui/package-lock.json
#	package-lock.json
@deepakprabhakara
renamed pdf chat to document chat
d91afbf
@deepakprabhakara
more tweaks
4b4480f
@deepakprabhakara
send baseURL as query param
e258a16
@deepakprabhakara
need to encode baseURL in the product param
82aa5d3
@deepakprabhakara
audience and issuer is fixed to urn:boxyhq
bffd3b5
@deepakprabhakara
Merge branch 'release' into llm-policies-ui-schema
cbdc867 
# Conflicts:
#	package-lock.json
#	package.json
@deepakprabhakara
tenant is not needed for jwt
3bfb63b
@deepakprabhakara
cleanup (#3402)
8ab952e 
* cleanup

* cleanup of npm packages

* more cleanup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@deepakprabhakara deepakprabhakara deepakprabhakara left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pi1814 @deepakprabhakara @niwsa @ukrocks007