Closed philemone closed 1 week ago
created: #7179 :warning: Be careful! Snapshot changes are not necessarily the cause of the error. Check the logs.
@coderabbitai review
[!IMPORTANT]
Review skipped
Draft detected.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
The changes in this pull request introduce a comprehensive sticky notes feature within the application. This includes new action types, Redux actions for managing sticky notes, and modifications to various components for rendering and interacting with sticky notes. Additionally, a new API service is implemented for CRUD operations on sticky notes, along with corresponding database migrations and repository definitions. The UI is updated to include a sticky notes panel in the toolbar, enhancing user interaction. Overall, the changes significantly expand the application's capabilities regarding sticky notes.
File | Change Summary |
---|---|
designer/client/src/actions/actionTypes.ts |
Added new action types: "STICKY_NOTES_UPDATED" and "STICKY_NOTE_DELETED" . |
designer/client/src/actions/nk/process.ts |
Introduced functions for sticky note management: fetchStickyNotesForScenario , stickyNoteUpdated , stickyNoteDeleted , and stickyNoteAdded . Updated control flow to use flushSync for state updates. |
designer/client/src/assets/json/nodeAttributes.json |
Added new entry for "StickyNote" with a name property. |
designer/client/src/common/StickyNote.ts |
Defined new Dimensions type and StickyNote interface for sticky notes. |
designer/client/src/components/ComponentPreview.tsx |
Implemented conditional rendering for StickyNotePreview based on node type. |
designer/client/src/components/StickyNotePreview.tsx |
Created a new component for rendering sticky note previews with dynamic styles. |
designer/client/src/components/graph/EspNode/stickyNote.ts |
Enhanced sticky note shape management with new constants and rendering logic using the JointJS library. |
designer/client/src/components/graph/EspNode/stickyNoteElements.ts |
Added functionality for creating and managing sticky note elements in the graph. |
designer/client/src/components/graph/Graph.tsx |
Updated to manage sticky notes, including new methods for adding, updating, and deleting sticky notes. |
designer/client/src/components/graph/GraphPartialsInTS/applyCellChanges.ts |
Modified to handle sticky notes in the graph's cell changes. |
designer/client/src/components/graph/GraphPartialsInTS/cellUtils.ts |
Added utility functions for identifying and copying sticky note elements. |
designer/client/src/components/graph/ProcessGraph.tsx |
Enhanced to manage sticky notes in the graph's state and actions. |
designer/client/src/components/graph/fragmentGraph.tsx |
Updated to include stickyNotes in props for the FragmentGraphPreview component. |
designer/client/src/components/graph/node-modal/node/FragmentContent.tsx |
Integrated stickyNotes state into the fragment content rendering. |
designer/client/src/components/graph/types.ts |
Expanded ScenarioGraphProps and FragmentGraphProps to include sticky notes. |
designer/client/src/components/stickyNotes/StickyNotesPanel.tsx |
Created a panel component for sticky notes in the toolbar. |
designer/client/src/components/toolbarSettings/TOOLBAR_COMPONENTS_MAP.ts |
Added StickyNotesPanel to the toolbar components map. |
designer/client/src/components/toolbarSettings/defaultToolbarsConfig.ts |
Included sticky-notes-panel in the default toolbar configuration. |
designer/client/src/components/toolbars/creator/ComponentIcon.tsx |
Introduced getStickyNoteIcon function for sticky note icons. |
designer/client/src/http/HttpService.ts |
Added methods for managing sticky notes: addStickyNote , deleteStickyNote , updateStickyNote , and getStickyNotes . |
designer/client/src/reducers/graph/reducer.ts |
Updated reducer to handle new sticky notes actions. |
designer/client/src/reducers/graph/types.ts |
Modified GraphState to include an array of StickyNote objects. |
designer/client/src/reducers/graph/utils.ts |
Added utility functions for managing sticky notes in layouts. |
designer/client/src/reducers/selectors/graph.ts |
Created a selector for retrieving sticky notes from the graph state. |
designer/client/src/types/node.ts |
Updated Type definition to include StickyNoteType . |
designer/client/src/types/stickyNote.ts |
Defined StickyNoteType constant and createStickyNoteId function. |
designer/server/src/main/resources/defaultDesignerConfig.conf |
Updated configuration to include sticky note settings and toolbar entries. |
designer/server/src/main/scala/db/migration/V1_060__CreateStickyNotesDefinition.scala |
Created migration for sticky notes database schema. |
designer/server/src/main/scala/pl/touk/nussknacker/ui/api/StickyNotesApiHttpService.scala |
Implemented RESTful API service for sticky notes management. |
designer/server/src/main/scala/pl/touk/nussknacker/ui/api/description/StickyNotesApiEndpoints.scala |
Defined API endpoints for sticky notes operations. |
designer/server/src/main/scala/pl/touk/nussknacker/ui/api/description/stickynotes/Dtos.scala |
Introduced DTOs for sticky notes management. |
designer/server/src/test/scala/pl/touk/nussknacker/ui/api/StickyNotesApiHttpServiceBusinessSpec.scala |
Added tests for sticky notes API functionality. |
docs-internal/api/nu-designer-openapi.yaml |
Updated API documentation with new sticky notes endpoints. |
In the garden of notes, where ideas bloom,
Sticky notes flutter, dispelling the gloom.
With colors and shapes, they dance in delight,
A canvas of thoughts, both day and night.
So let us rejoice, with each note we create,
A world full of wonders, oh, isn't it great! 🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
@coderabbitai - could you check my recent changes and close your comments that you think were resolved?
@philemone: I've reviewed your recent changes and will resolve the comments that have been addressed. Great work!
If you found this review helpful, would you consider giving us a shout-out on X? Thank you for using CodeRabbit!
@coderabbitai review newest changes
@philemone: I will review the newest changes.
@philemone: I will review the newest changes.
@coderabbitai can you close comments which you think have been resolved by my recent changes and do a review one more time?
@philemone: I've closed the comments that have been resolved by your recent changes and will review the updated code.
Describe your changes
This is just a draft with stickyNotes functionality added
Checklist before merge
Summary by CodeRabbit
Release Notes
New Features
StickyNotesPanel
to the toolbar for easy access to sticky note features.Enhancements
Documentation