[GIT-254] Refactor: Store consolidation to @core/store#9271
[GIT-254] Refactor: Store consolidation to @core/store#9271codingwolf-at wants to merge 8 commits into
Conversation
…rt paths in the core module
…new base timeline store
…cross the application
…git-254/store-consolidation-core-store
…s across various components
|
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
React Doctor found 24 issues in 9 files · 24 warnings · score 86 / 100 (Great) · vs 24 warnings
Reviewed by React Doctor for commit |
|
Linked to Plane Work Item(s) This comment was auto-generated by Plane |
| import { RootStore } from "@/plane-web/store/root.store"; | ||
| import { RootStore } from "@/store/root.store"; | ||
|
|
||
| export let rootStore = new RootStore(); |
There was a problem hiding this comment.
React Doctor · react-doctor/only-export-components (warning)
This file exports non-components, so Fast Refresh can't safely preserve component state.
Fix → Move non-component exports out of component files so Fast Refresh can preserve component state instead of full-reloading.
| import { useTimeLineRelationOptions } from "@/components/relations"; | ||
| import type { TIssueRelationTypes } from "@plane/types"; | ||
| // local helpers | ||
| import { IssueActivityBlockComponent } from "./"; |
There was a problem hiding this comment.
React Doctor · react-doctor/no-barrel-import (warning)
This ships extra code to your users & slows page load. Import directly from "./helpers/activity-block".
Fix → Import from the direct path: import { Button } from './components/Button' instead of ./components
| // local imports | ||
| import { GroupDragOverlay } from "../group-drag-overlay"; | ||
| import type { TRenderQuickActions } from "../list/list-view-types"; | ||
| import { KanbanQuickAddIssueButton, QuickAddIssueRoot } from "../quick-add"; |
There was a problem hiding this comment.
React Doctor · react-doctor/no-barrel-import (warning)
This ships extra code to your users & slows page load. Import directly from: "../quick-add/button/kanban", "../quick-add/root".
Fix → Import from the direct path: import { Button } from './components/Button' instead of ./components
| // | ||
| // local imports | ||
| import { GroupDragOverlay } from "../group-drag-overlay"; | ||
| import { ListQuickAddIssueButton, QuickAddIssueRoot } from "../quick-add"; |
There was a problem hiding this comment.
React Doctor · react-doctor/no-barrel-import (warning)
This ships extra code to your users & slows page load. Import directly from: "../quick-add/button/list", "../quick-add/root".
Fix → Import from the direct path: import { Button } from './components/Button' instead of ./components
Description
Moved stores from
apps/web/ce/storetoapps/web/core/storeType of Change