optimistic ui with jobs table, half baked (need to test & work on design)

[CodeTorso]

Optimistic UI with Jobs Table, Half Baked

Overview

This pull request introduces significant enhancements to the Memories Page component within the web application. The primary focus is on integrating a new data structure called Job into the existing MemoriesAndSpaces object, which previously only contained Content and StoredSpace data. This update aims to improve the user interface by displaying unsaved job data alongside saved memories.

Changes

• New Features:

  • Added a new unsavedMemories field in the memoriesAndSpaces object, which is an array of Job objects.
  • Introduced an "optimistic UI" feature that displays unsaved job data next to saved memories.
  • Added a new h1 element to display the JSON representation of the unsavedMemories array.

• Refactoring:

  • Merged the memories and unsavedMemories arrays into a single unifiedItems array for rendering purposes.
  • Updated the type definition of the memoriesAndSpaces object to include the new unsavedMemories field.
  • Moved the logic for adding memories from the Menu component to a new Content component to enhance code organization and maintainability.

• Other Changes:

  • Imported the Job type from the @repo/db/schema module to ensure type consistency.
  • Modified the getAllUserMemoriesAndSpaces fetcher to include unsaved memories in the response.
  • Added a new jobs table to the database schema to support the new functionality.

✨ Generated with love by Kaizen ❤️

Original Description

None

[kaizen-bot[bot]]

Code Review

❗ Attention Required: This PR has potential issues. 🚨

Direct use of JSON.stringify in JSX

Avoid using `JSON.stringify` directly in JSX for rendering as it can lead to unexpected output or errors. It's generally preferable to extract the desired values and format them appropriately for display.

Potential Solution: 1. Extract the relevant information from `memoriesAndSpaces.unSavedMemories`. 2. Format the extracted data into a user-friendly string or use JSX elements for structured display.

apps/web/app/(dash)/(memories)/content.tsx | 235 - 235

reason_for_request: Directly using `JSON.stringify` in JSX can lead to unexpected output, especially when dealing with nested objects or arrays. It might not always render as intended and can hinder the user experience. Additionally, if the data contains circular references, it can cause runtime errors.

level: [critical] , severity: [10]

✨ Generated with love by Kaizen ❤️

Useful Commands

• Feedback: Reply with !feedback [your message]

• Ask PR: Reply with !ask-pr [your question]

• Review: Reply with !review

[Dhravya]

Excited for this

[CodeTorso]

Excited for this

this implementation sucks, should wait for the backend change, till then will make a workaround in the next pr that is #232