chore: add Seata Saga designer as an external project

bpmn-io / diagram-js

A toolbox for displaying and modifying diagrams on the web.

MIT License

1.69k stars 419 forks source link

chore: add Seata Saga designer as an external project #852

Closed ptyin closed 8 months ago

ptyin commented 8 months ago

Hi, @nikku

Thanks for your great work. Apache Seata (Incubator) is a distributed transaction framework. And recently, we have used diagram-js as the backbone to refactor our new version of Saga Designer (a visual tool to orchestrate Saga pattern transaction).

This PR aims to provide an external link in the README.

nikku commented 8 months ago

@ptyin Nice to see you're using diagram-js to visualize things. Any particular reason you closed the PR again?

ptyin commented 8 months ago

@ptyin Nice to see you're using diagram-js to visualize things. Any particular reason you closed the PR again?

Waiting for feedback for a week made me aware that it might be considered impolite to submit a pull request without prior discussion or issue. In this case, what are your thoughts on the idea of adding it?

nikku commented 8 months ago

We're happy to add this if it is a good showcase for our libraries. Consider re-opening.

We're candid, too. So if we feel like a contribution needs prior discussion we're going to voice it.