-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
V3 make workflowdefinition have readonly #4068
V3 make workflowdefinition have readonly #4068
Conversation
import { ActivityNodeShape } from './shapes'; | ||
import { Activity, ActivityDeletedArgs, ActivityDescriptor, ActivitySelectedArgs, ChildActivitySelectedArgs, ContainerSelectedArgs, EditChildActivityArgs, GraphUpdatedArgs, WorkflowUpdatedArgs } from '../../models'; | ||
import { createGraph } from './graph-factory'; | ||
import { AddActivityArgs, Connection, Flowchart, FlowchartModel, FlowchartPathItem, LayoutDirection, RenameActivityArgs, UpdateActivityArgs } from './models'; | ||
import { NodeFactory } from "./node-factory"; | ||
import { Container } from "typedi"; | ||
import { createActivityLookup, EventBus, flatten, PortProviderRegistry, walkActivities } from "../../services"; | ||
import { ConnectionCreatedEventArgs, FlowchartEvents } from "./events"; | ||
import { ContextMenuAnchorPoint, ContextMenuItem } from "../../components/shared/context-menu/models"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Many changes in this PR relate to different code style settings between your and my IDE. Although I like your styling better, it would be best if you can adjust your settings (ideally, via some config file part of this repo) and update your PR to only include the relevant changes to support readonly workflows.
This PR looks good, I have tested it too, and it works well. If you could handle the style changes, then that would be perfect. |
…thub.com/herblinernexxbizz/elsa-core into v3_make_workflowdefinition_have_readonly
make it possible to have 'readonly' workflows