UI tab backup #319

milaiwi · 2024-07-17T15:12:36Z

Added

Tab Support
Added padding on File List
Added padding on TextEditor content (can be disabled in settings)
Moved General Page to the top of the Settings (should always be first)
Changed coloring around
Added EmptyPage when no file is opened

Let me know if any bugs

milaiwi · 2024-07-17T15:19:03Z

Is there a way to run the linter locally?

greptile-apps

PR Summary

Added IPC handlers for managing editor flex center settings and open tabs (electron/main/electron-store/ipcHandlers.ts)
Introduced new Tab type and updated StoreSchema and StoreKeys enums for tab management (electron/main/electron-store/storeConfig.ts)
Added EmptyPage component for when no file is selected (src/components/EmptyPage.tsx)
Introduced TabProvider for managing tabs within the application (src/components/Providers/TabProvider.tsx)
Updated various components to include widthType prop for dynamic modal sizing (src/components/Common/Modal.tsx, src/components/File/NewDirectory.tsx, src/components/File/NewNote.tsx, src/components/File/RenameDirectory.tsx, src/components/File/RenameNote.tsx, src/components/Flashcard/FlashcardMenuModal.tsx, src/components/Flashcard/FlashcardReviewModal.tsx)

_{29 file(s) reviewed, 13 comment(s)}
_{Edit PR Review Bot Settings}

electron/preload/index.ts

greptile-apps · 2024-07-17T15:15:40Z

src/components/Common/Modal.tsx

@@ -6,17 +6,66 @@ interface ModalProps {
  children: React.ReactNode
  hideCloseButton?: boolean
  tailwindStylesOnBackground?: string
+  widthType: number


Style: Consider using a string type for widthType instead of number for better type safety.

greptile-apps · 2024-07-17T15:15:41Z

src/components/Common/Modal.tsx

@@ -40,7 +89,7 @@ const ReorModal: React.FC<ModalProps> = ({
    >
      <div
        ref={modalRef}
-        className="flex max-w-4xl  flex-col items-center justify-center rounded-lg border border-solid border-gray-600 bg-dark-gray-c-three shadow-xl"
+        className={`bg-dark-gray-c-three rounded-lg shadow-xl w-${widthClass} max-w-6xl  flex flex-col justify-center items-center border-solid border-gray-600 border`}


Logic: Using template literals for class names can lead to unexpected results if widthClass is not a valid Tailwind class.

greptile-apps · 2024-07-17T15:15:59Z

src/components/Common/ResizableComponent.tsx

+        const deltaWidth = resizeSide === "left" ? -e.movementX : e.movementX;
+        setWidth((prevWidth) => {
+          const newWidth = prevWidth + deltaWidth;
+          if (newWidth > setMaximumWidthCap) return setMaximumWidthCap;


Logic: Ensure the new width does not exceed the maximum width cap.

greptile-apps · 2024-07-17T15:16:00Z

src/components/Common/ResizableComponent.tsx

+        setWidth((prevWidth) => {
+          const newWidth = prevWidth + deltaWidth;
+          if (newWidth > setMaximumWidthCap) return setMaximumWidthCap;
+          if (newWidth < 50) return 50;


Logic: Ensure the new width does not fall below the minimum width of 50.

greptile-apps · 2024-07-17T15:16:53Z

src/components/Editor/EditorManager.tsx

-      />
+      <div
+        className={`${
+          editorFlex ? "flex justify-center items-center h-screen p-4" : ""


Logic: Potential issue: h-screen class might cause layout issues if the editor content exceeds the viewport height.

greptile-apps · 2024-07-17T15:16:54Z

src/components/Editor/EditorManager.tsx

-      />
+      <div
+        className={`${
+          editorFlex ? "flex justify-center items-center h-screen p-4" : ""


Style: Consider removing h-screen to avoid potential overflow issues.

greptile-apps · 2024-07-17T15:17:09Z

src/components/EmptyPage.tsx

+const EmptyPage = ({ vaultDirectory }) => {
+  const handleCreateFile = () => {
+    window.electronUtils.showCreateFileModal(vaultDirectory);
+    console.log("Attemtping to create new file!");


Spelling: Typo in 'Attemtping'.

Suggested change

console.log("Attemtping to create new file!");

console.log("Attempting to create new file!");

greptile-apps · 2024-07-17T15:17:10Z

src/components/EmptyPage.tsx

+  };
+
+  const handleCreateFolder = () => {
+    window.electronUtils.showCreateDirectoryModal(vaultDirectory);


Style: Consider adding error handling for showCreateDirectoryModal.

greptile-apps · 2024-07-17T15:20:44Z

src/components/Providers/TabProvider.tsx

+  const syncTabsWithBackend = async (action, args) => {
+    await window.electronStore.setCurrentOpenFiles(action, args);


Style: Consider adding error handling for the async function to manage potential failures.

milaiwi · 2024-07-21T03:43:36Z

Cleaned up the code. Had to make a change in electron/main/llm/ipcHandlers.ts that I am not too confident in yet. It mainly just ensures that the argument we pass in streamingResponse has the same type has the one accepted in our model creation.

greptile-apps

PR Summary

(updates since last review)

The pull request introduces several UI enhancements and tab management functionalities, along with various code refactorings and stylistic adjustments.

Tab Management: Added handlers for managing tabs in electron/main/electron-store/ipcHandlers.ts and updated TabProvider in src/components/Providers/TabProvider.tsx.
UI Enhancements: Added padding to File List and TextEditor content, and introduced EmptyPage for when no file is opened (src/components/EmptyPage.tsx).
Settings Reorganization: Moved General Page to the top of the settings and updated GeneralSettings.tsx for better readability.
Spell Check: Integrated nspell library for spell-checking (src/utils/strings.ts) and updated related IPC handlers in electron/preload/index.ts.
Stylistic Changes: Removed semicolons and adjusted import orders for consistency across multiple files (electron/main/electron-store/storeConfig.ts, electron/main/electron-utils/ipcHandlers.ts).

No significant security issues were introduced, but thorough testing is recommended to ensure the new functionalities work as expected.

_{37 file(s) reviewed, 5 comment(s)}
_{Edit PR Review Bot Settings}

greptile-apps · 2024-07-21T03:42:02Z

electron/main/electron-store/ipcHandlers.ts

+    const removeTab = ({ tabId, idx, newIndex }: { tabId: string; idx: number; newIndex: number }) => {
+      openTabs[idx].lastAccessed = false
+      openTabs[newIndex].lastAccessed = true
+      const updatedTabs = openTabs.filter((tab) => tab.id !== tabId)
+      store.set(StoreKeys.OpenTabs, updatedTabs)


Logic: Potential issue: Ensure idx and newIndex are valid indices to avoid runtime errors.

Could we add the checking the bot recommends here @milaiwi ?

Yeah, I'll go ahead and add it, but either way I'm pretty sure it is not possible for idx and newIndex not to be valid since tabId is guaranteed to exist which means idx and newIndex must exist.

greptile-apps · 2024-07-21T03:47:59Z

src/components/Common/Modal.tsx

@@ -40,7 +83,7 @@ const ReorModal: React.FC<ModalProps> = ({
    >
      <div
        ref={modalRef}
-        className="flex max-w-4xl  flex-col items-center justify-center rounded-lg border border-solid border-gray-600 bg-dark-gray-c-three shadow-xl"
+        className={`w-${widthClass} flex max-w-6xl flex-col items-center justify-center rounded-lg border border-solid border-gray-600 bg-dark-gray-c-three shadow-xl`}


Logic: Using template literals for class names can lead to unexpected results if widthClass is not a valid Tailwind class.

greptile-apps · 2024-07-21T03:48:28Z

src/components/Common/TitleBar.tsx

+        openTabs.forEach((tab: Tab) => {
+          if (tab.lastAccessed) {
+            setOpenedLastAccess(true)
+            openFileAndOpenEditor(tab.filePath)
+          }
+        })


Logic: Potential issue: openFileAndOpenEditor might be called multiple times if multiple tabs have lastAccessed set.

Agree with bot. Perhaps we should break out of the loop as soon as we open a file based on tab.lastAccessed?

Not sure if you can break out of a forEach since it is just an underlying recursive function. I'll change it to use .some instead.

src/components/Providers/TabProvider.tsx

src/utils/strings.ts

milaiwi · 2024-07-24T02:36:06Z

@samlhuillier That should be the final commit. Take a look at this just cleaned up sidebars and UI on file sidebar. LMK what you think!

greptile-apps

PR Summary

(updates since last review)

The pull request introduces several UI enhancements, tab management functionalities, and code refactorings.

ResizableComponent.tsx: Removed onResize prop and max width cap logic, potentially introducing width management issues.
TitleBar.tsx: Removed sidebarWidth prop and added logic for handling the last accessed tab, enhancing usability.
EditorManager.tsx: Added opacity style, modified padding, and scrollbar visibility, potentially causing layout issues with h-screen.
MainPage.tsx: Removed resizableWidth state and added overflow-x-hidden class, simplifying state management and preventing horizontal overflow.
global.css: Set scrollbar width to 0px, improving aesthetics but potentially affecting usability.

_{5 file(s) reviewed, 2 comment(s)}
_{Edit PR Review Bot Settings}

greptile-apps

PR Summary

(updates since last review)

The pull request introduces UI enhancements, including tab support, padding adjustments, and layout improvements.

src/components/Editor/EditorManager.tsx: Added overflow handling to EditorContent for better scrollability and layout management.
src/components/Editor/EditorManager.tsx: Adjusted padding for TextEditor content, configurable via settings.
Settings: Moved General Page to the top for better accessibility.
UI: Introduced EmptyPage for scenarios with no open files.
Styling: Updated coloring and padding for improved aesthetics and usability.

_{1 file(s) reviewed, no comment(s)}
_{Edit PR Review Bot Settings}

samlhuillier · 2024-07-29T07:26:58Z

Hey @milaiwi apologies for the delay. I really like the tabs experience! Super wicked piece of work! I'll start reviewing it and edit this comment re changes and suggestions.

The spellchecking (where it underlines incorrectly spelt words) seems to have been lost. I'd propose given the size of this PR, we split off the suggested words feature into a separate PR :)
I've merged in the branch which should allow github workflows on this branch so you can see errors/lint etc.
The back and forth button styling is broken with tabs:

If I hit enter and overflow the editor and then start typing at the bottom, some weird scrollbars appear:

On styling:
I like the new grey look on the icons, with the accent color just being a lighter white. However, I think it looks slightly too subdued:

- I also think the padding is great in the editor but not so great in the file sidebar where it feels like it's harder to see file names and we're in quite precious screen real estate there

greptile-apps

PR Summary

(updates since last review)

The pull request introduces UI enhancements, including tab support, padding adjustments, and layout improvements.

scripts/notarize.js: Added a console log statement to indicate the start of the notarization process.
src/components/MainPage.tsx: Integrated EmptyPage component for scenarios with no open files.
src/components/Settings/Settings.tsx: Moved General Page to the top for better accessibility.
src/components/Editor/EditorManager.tsx: Adjusted padding for TextEditor content, configurable via settings.
src/components/Sidebars/IconsSidebar.tsx: Updated sidebar icons and added new modal triggers for better user interaction.

_{1 file(s) reviewed, no comment(s)}
_{Edit PR Review Bot Settings}

This reverts commit 3402938.

greptile-apps

PR Summary

(updates since last review)

The pull request introduces several UI enhancements and improvements to the notarization process.

scripts/notarize.js: Enhanced logging and error handling, including directory tree printing for better debugging.
src/components/MainPage.tsx: Integrated EmptyPage component for scenarios with no open files.
src/components/Settings/Settings.tsx: Moved General Page to the top for better accessibility.
src/components/Editor/EditorManager.tsx: Added padding for TextEditor content, configurable via settings.
src/components/Sidebars/IconsSidebar.tsx: Updated sidebar icons and added new modal triggers for better user interaction.

_{1 file(s) reviewed, no comment(s)}
_{Edit PR Review Bot Settings}

greptile-apps

PR Summary

(updates since last review)

The pull request introduces several UI enhancements and improvements to the notarization process.

.prettierrc: Updated endOfLine to lf for consistent line endings across OS.
package.json: Renamed postbuild script to notarize for clarity.
scripts/notarize.js: Added logging for better feedback during notarization.
src/components/MainPage.tsx: Integrated EmptyPage component for scenarios with no open files.
src/components/Settings/Settings.tsx: Moved General Page to the top for better accessibility.

_{3 file(s) reviewed, no comment(s)}
_{Edit PR Review Bot Settings}

samlhuillier · 2024-07-29T14:12:00Z

Is there a way to run the linter locally?

Yes run the linter like this:

npm run lint

making sure you've done an npm i

samlhuillier · 2024-07-29T05:43:53Z

src/components/Common/TitleBar.tsx

+  // Note: Do not put dependency on addTab or else removeTab does not work properly.
+  // Typically you would define addTab inside the useEffect and then call it but since
+  // we are using it inside a useContext we can remove it
+  /* eslint-disable */


Could we not do something like:

const memoizedAddTab = useCallback((filePath) => { addTab(filePath); }, [addTab]); useEffect(() => { if (!currentFilePath) return; memoizedAddTab(currentFilePath); }, [currentFilePath, memoizedAddTab]);

addTab already uses useCallback, so this would essentially be the same thing. I'll take a deeper look on how to fix this

samlhuillier · 2024-07-29T06:49:54Z

public/dictionaries/en_US/en_US.aff

What was the reason for adding this? Until now, we've kept spellcheck fairly simple by just leveraging the user system spellcheck. Perhaps we should add this for every language if you think it's a value add...

It's needed for replaceWord. It takes the word that the spellcheck highlights as wrong and finds the correct word it should be. I don't think tiptap has a built-in replaceWord so we have to make a custom one.

Is there no library for this either? Because I think having these big dictionary files is not something we should have in the codebase nor something we should maintain...

samlhuillier · 2024-07-29T06:53:47Z

src/components/Common/Modal.tsx

+  newEmbeddingModel: '[500px]',
+  localLLMSetting: '[500px]',
+  remoteLLMSetting: '[500px]',
+  indexingProgress: '[850px]',


This stuff with mapping modal types to a width was removed a few commits back in favour of just allowing the child component to dictate the width, which I think leads to nice clean code 😅

Do you have strong opinions on needing to add this back? Can't you just set the width in the child component?

samlhuillier · 2024-07-29T08:00:50Z

src/components/Common/ResizableComponent.tsx

@@ -23,7 +23,7 @@ const ResizableComponent: React.FC<ResizableComponentProps> = ({
    (e: MouseEvent) => {
      if (isDragging) {
        const deltaWidth = resizeSide === 'left' ? -e.movementX : e.movementX
-        setWidth((prevWidth) => prevWidth + deltaWidth)
+        setWidth(deltaWidth)


Why did you add this? It breaks dragging of the sidebars...

Screen.Recording.2024-07-29.at.09.29.32.mov

Originally I made the tabs move depending on where the sidebar was located, but ended up scrapping that and just keeping it at a fixed location on the top left corner. I guess I didn't clean it up good when I scrapped. It's fixed now.

samlhuillier · 2024-07-29T09:13:55Z

electron/main/electron-store/ipcHandlers.ts

@@ -114,6 +115,13 @@ export const registerStoreHandlers = (store: Store<StoreSchema>, windowsManager:

  ipcMain.handle('get-sb-compact', () => store.get(StoreKeys.IsSBCompact))

+  ipcMain.handle('get-editor-flex-center', () => store.get(StoreKeys.EditorFlexCenter))


Could you explain what this feature does? I tried it in settings and didn't notice any changes in the editor nor anywhere else

Update: I now see a slight difference in padding - do you have strong opinions on adding this feature? I think it wouldn't really be clear to the user what "Editor Flex" is + I think we should just have a standard default for things like padding so that we reduce user load in having to make decisions

The difference in padding depends on your screen size. On a Mac or small monitor, the difference is negligible. However, on a standard monitor (21–24 inches), like the one I use, it significantly impacts the UI. I can try to change the name and subcaption to make it more clearer.

I think we should just have a standard default for things like padding so that we reduce user load in having to make decisions

I don't think this impacts user load that much.

Ah ok sounds good. Why don't we even say something in the subcaption like "this is useful if you have a big monitor"?

Or perhaps there's even some kind of standard tailwind/css config for this kind of thing where you adjust padding based on screen size?

samlhuillier · 2024-07-29T14:30:22Z

src/components/File/hooks/use-file-by-filepath.ts

@@ -185,7 +185,7 @@ const useFileByFilepath = () => {
      editor.setOptions({
        editorProps: {
          attributes: {
-            spellcheck: spellCheckEnabled,
+            spellcheck: spellCheckEnabled.toString(),


Why this change? I think this is what is causing spellcheck to not work on my machine...

Originally, the way spellCheck was stored and handled in the IPC was by maintaining and storing a state that had two values: "False" or "True". On the slider we converted it to a boolean, but stored it as a string. I thought this was weird so I changed it to be a boolean and converted it to a string when needed (in the case highlighted).

samlhuillier · 2024-07-29T15:59:13Z

src/components/Settings/GeneralSections.tsx

-}
-
-const CreateAppearanceSection: React.FC = () => {
+export const CreateAppearanceSection = () => {


Why is this called CreateAppearanceSection? It should be called AppearanceSection given that it's a component not a function. Same with CreateEditorSection

samlhuillier · 2024-07-29T16:06:13Z

src/components/Settings/Settings.tsx

+  TextGenerationTab = 'textGeneration',
+  // RAG = "RAG",
+  ANALYTICSTab = 'analytics',
+  ChunkSizeTab = 'chunkSize',


Why introduce new tabs that don't exist? Also nit but why is ANALYTICSTab all capitalised?

Why introduce new tabs that don't exist?

What do you mean?

Also nit but why is ANALYTICSTab all capitalised?

Fixed

samlhuillier · 2024-07-29T16:18:46Z

src/components/Sidebars/IconsSidebar.tsx

@@ -116,25 +130,29 @@ const IconsSidebar: React.FC<IconsSidebarProps> = ({ openRelativePath, sidebarSh
        onClick={() => setIsNewNoteModalOpen(true)}
      >
        <div className="flex size-4/5 items-center justify-center rounded hover:bg-neutral-700">
-          <VscNewFile className="text-gray-200" size={22} title="New Note" />
+          <IconContext.Provider value={newFileContextValue}>


Are these IconContext.Providers necessary? They seem to introduce a lot of baggage code when we could just pass the colour in directly no?

Yeah, I think you're right here. I ended up replacing all IconContext.Providers to just color with the appropriate ternaries.

samlhuillier · 2024-07-29T16:20:58Z

src/components/Sidebars/TabSidebar.tsx

+          id="titleBarFileNavigator"
+          className="ml-1 flex h-[28px] cursor-pointer items-center justify-center px-2 text-white hover:rounded-md hover:bg-dark-gray-c-three"
+          onClick={() => {
+            window.electronUtils.showCreateFileModal('')


Again as above - this functionality should be passed in either as a prop or a provider so that we avoid unnecessary extra calls to the backend

I'll consider the provider, but I disagree with the prop suggestion. In reality, this won't be called frequently enough to worry about backend costs. However, making it a prop would make the code messier and less readable.

samlhuillier · 2024-07-29T16:26:20Z

electron/main/llm/ipcHandlers.ts

@@ -44,13 +44,18 @@ export const registerLLMSessionHandlers = (store: Store<StoreSchema>) => {
        event.sender.send('anthropicTokenStream', chatHistory.id, chunk)
      }

+      const transformedChatHistory: ChatCompletionMessageParam[] = chatHistory.displayableChatHistory.map((message) => {


What was the reason for this change out of interest?

Something to note is that we are going to move the LLM calling logic all to the renderer process, it'll make it so much easier to not have to deal with all this messy logic with chunks being passed via IPC 😅

milaiwi · 2024-07-29T23:08:24Z

I'll take a look at this again tonight and answer the questions

milaiwi · 2024-08-09T05:11:44Z

Closing this and opening a new one. @samlhuillier Check #339

mohamedilaiwi and others added 23 commits June 17, 2024 23:44

Starting adding tab support, but cannot extend directly from history.

5afc528

Added tab support with highlighting

90bc82a

Making LLMSession more Generic

94295d0

parsing bug with chat and format

87a6ae6

Need to resolve answering questions properly

25e9dd0

Merge branch 'create-dir-files-on-rightclick'

5f59c9b

Merge branch 'main' into add-tab-support

dde3f79

Added new tab support

5b7665e

Refined some code

f254e94

Refined UI even more

8eb8e84

Merging UI and Tab Support

47667e2

Merge branch 'add-tab-support' into ui-tab-support

cee70b6

Merged with add-tab-support

7c9a9b9

Added Tab Support with more UI Changes

4f34822

More changes to UI. Added Scrolling

13ae394

Tab better UI.Does not display + on null tabs. Shifts tabs on delete

9b456f5

Temp save

b1e17c2

Mostly complete. Need to add name dropping on hover.

b62e4e4

Added displaying tab path when hovering

bc51bd5

Complete

2e23e7d

Changes needed

a9ce1bd

Completely fixed

b79d432

Complete with fixed merge conflicts

b5f38e9

greptile-apps bot reviewed Jul 17, 2024

View reviewed changes

mohamedilaiwi added 2 commits July 20, 2024 22:34

replaceWord added. cleaned up codebase

6f9c6a4

Added replaceWord. Cleaned up codebase

df16e5d

greptile-apps bot reviewed Jul 21, 2024

View reviewed changes

cleaned up sidebars. Final commit

a1702d5

greptile-apps bot reviewed Jul 24, 2024

View reviewed changes

Complete

16589ff

greptile-apps bot reviewed Jul 24, 2024

View reviewed changes

samlhuillier added 2 commits July 29, 2024 08:43

force line ending lf

32a86f4

test no notarize unless on main branch

66ba83c

reorproject deleted a comment from greptile-apps bot Jul 29, 2024

update release notarize script

3402938

greptile-apps bot reviewed Jul 29, 2024

View reviewed changes

Revert "update release notarize script"

2bc39af

This reverts commit 3402938.

greptile-apps bot reviewed Jul 29, 2024

View reviewed changes

samlhuillier added 2 commits July 29, 2024 10:18

notarize on release intentional

d6be068

Merge branch 'fix/test-line-endings' into ui-tab-backup

83478f3

greptile-apps bot reviewed Jul 29, 2024

View reviewed changes

reorproject deleted a comment from greptile-apps bot Jul 29, 2024

samlhuillier requested changes Jul 29, 2024

View reviewed changes

samlhuillier reviewed Jul 29, 2024

View reviewed changes

milaiwi closed this Aug 9, 2024

milaiwi deleted the ui-tab-backup branch August 11, 2024 06:08

	console.log("Attemtping to create new file!");
	console.log("Attempting to create new file!");

		const syncTabsWithBackend = async (action, args) => {
		await window.electronStore.setCurrentOpenFiles(action, args);

		@@ -114,6 +115,13 @@ export const registerStoreHandlers = (store: Store<StoreSchema>, windowsManager:

		ipcMain.handle('get-sb-compact', () => store.get(StoreKeys.IsSBCompact))

		ipcMain.handle('get-editor-flex-center', () => store.get(StoreKeys.EditorFlexCenter))

UI tab backup #319

UI tab backup #319

Conversation

milaiwi commented Jul 17, 2024 • edited Loading

milaiwi commented Jul 17, 2024

greptile-apps bot left a comment

Choose a reason for hiding this comment

PR Summary

greptile-apps bot Jul 17, 2024

Choose a reason for hiding this comment

greptile-apps bot Jul 17, 2024

Choose a reason for hiding this comment

greptile-apps bot Jul 17, 2024

Choose a reason for hiding this comment

greptile-apps bot Jul 17, 2024

Choose a reason for hiding this comment

greptile-apps bot Jul 17, 2024

Choose a reason for hiding this comment

greptile-apps bot Jul 17, 2024

Choose a reason for hiding this comment

greptile-apps bot Jul 17, 2024

Choose a reason for hiding this comment

greptile-apps bot Jul 17, 2024

Choose a reason for hiding this comment

greptile-apps bot Jul 17, 2024

Choose a reason for hiding this comment

milaiwi commented Jul 21, 2024

greptile-apps bot left a comment

Choose a reason for hiding this comment

PR Summary

greptile-apps bot Jul 21, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

milaiwi Jul 30, 2024 • edited Loading

Choose a reason for hiding this comment

greptile-apps bot Jul 21, 2024

Choose a reason for hiding this comment

greptile-apps bot Jul 21, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

milaiwi commented Jul 24, 2024

greptile-apps bot left a comment

Choose a reason for hiding this comment

PR Summary

greptile-apps bot left a comment

Choose a reason for hiding this comment

PR Summary

samlhuillier commented Jul 29, 2024 • edited Loading

greptile-apps bot left a comment

Choose a reason for hiding this comment

PR Summary

greptile-apps bot left a comment

Choose a reason for hiding this comment

PR Summary

greptile-apps bot left a comment

Choose a reason for hiding this comment

PR Summary

samlhuillier commented Jul 29, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

milaiwi Jul 30, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

samlhuillier Jul 31, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

milaiwi commented Jul 17, 2024 •

edited

Loading

milaiwi Jul 30, 2024 •

edited

Loading

samlhuillier commented Jul 29, 2024 •

edited

Loading

milaiwi Jul 30, 2024 •

edited

Loading

samlhuillier Jul 31, 2024 •

edited

Loading