Remove old todos and update todos with github user

cohere-ai · Apr 29, 2024 · 5589280 · 5589280
1 parent 51ddb20
commit 5589280
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 7 deletions.
diff --git a/src/interfaces/coral_web/src/components/Citations/CitationDocument.tsx b/src/interfaces/coral_web/src/components/Citations/CitationDocument.tsx
@@ -151,7 +151,6 @@ export const CitationDocument: React.FC<Props> = ({ document, isExpandable = fal
       {isExpanded && isExpandable && (
         <div className="flex flex-col">
           {getSnippet(30, 'line-clamp-3')}
-          {/* TODO(jessica): should we enable this/will we support this? */}
           <button
             className="self-end p-0 text-primary-900 transition-colors ease-in-out hover:text-primary-700"
             onClick={openFullSnippetModal}

diff --git a/src/interfaces/coral_web/src/components/Citations/CitationTextHighlighter.tsx b/src/interfaces/coral_web/src/components/Citations/CitationTextHighlighter.tsx
@@ -80,7 +80,6 @@ export const CitationTextHighlighter: React.FC<Props> = ({
             generationId={generationId}
             // Used to find the keyword to bold but since we don't have it yet here when the message is
             // still streaming in we can just ignore it for now instead of not showing the popup at all
-            // TODO(jessica): pass in cited text from citation event as a backup
             message={message?.originalText ?? ''}
           />
         ),

diff --git a/src/interfaces/coral_web/src/hooks/chat.ts b/src/interfaces/coral_web/src/hooks/chat.ts
@@ -294,7 +294,7 @@ export const useChat = (config?: { onSend?: (msg: string) => void }) => {
 
               const finalText = isRAGOn
                 ? replaceTextWithCitations(
-                    // TODO(jessica): temporarily use the text generated from the stream when MAX_TOKENS
+                    // TODO(@wujessica): temporarily use the text generated from the stream when MAX_TOKENS
                     // because the final response doesn't give us the full text yet. Note - this means that
                     // citations will only appear for the first 'block' of text generated.
                     data?.finish_reason === FinishReason.FINISH_REASON_MAX_TOKENS
@@ -309,7 +309,7 @@ export const useChat = (config?: { onSend?: (msg: string) => void }) => {
                 type: MessageType.BOT,
                 state: BotState.FULFILLED,
                 generationId,
-                // TODO(jessica): TEMPORARY - we don't pass citations for langchain multihop right now
+                // TODO(@wujessica): TEMPORARY - we don't pass citations for langchain multihop right now
                 // so we need to manually apply this fix. Otherwise, this comes for free when we call
                 // `replaceTextWithCitations`.
                 text: citations.length > 0 ? finalText : fixMarkdownImagesInText(finalText),

diff --git a/src/interfaces/coral_web/src/themes/cohereTheme.js b/src/interfaces/coral_web/src/themes/cohereTheme.js
@@ -1,11 +1,9 @@
 const defaultTheme = require('tailwindcss/defaultTheme');
 
-// any changes to the theme should be reflected into our `customTwMerge` in  `utils/cn.ts`
+// Any changes to the theme should be reflected into our `customTwMerge` in `utils/cn.ts`
 module.exports = {
   theme: {
     extend: {
-      // TODO(jessica): shift colors to `extend` for now so that using the old theme and nature theme
-      // merge nicely
       colors: {
         transparent: 'transparent',
         pureWhite: '#FFFFFF',