Xiangkun Yin

Results 31 comments of Xiangkun Yin

> Hi @ptyin , > > Do you have any existing similar PR so that I can review it? Some designer related PRs include: - #6169 - #6118 However, I...

> Just I want confirmation whether I am on right way or not > > Issue is we are not able to import the designer directly through json due to...

> Below are the some question before I need to start implementation ? > > 1. Are we asking user to add details in json if they are not present...

> Consider following example : > > { "Name": "StateMachine-qw7ufbv", "Comment": "This state machine is modeled by designer tools.", "Version": "0.0.1", "style": { "bounds": { "x": 200, "y": 200, "width":...

> Yeah, I got that . But consider same example : > > [StateMachine-qw7ufbv.json](https://github.com/apache/incubator-seata/files/14386779/StateMachine-qw7ufbv.json) > > In state "ServiceTask-6cc74ej" there are two CompensateState which are "ServiceTask-vezv7z5" and "ServiceTask-vhxci8q" , in...

> May I know what is going on? How can I merge my pull request to the target branch? According to the community [CONTRIBUTING.md](https://github.com/apache/incubator-seata/blob/2.x/CONTRIBUTING.md), you may not merge PR directly....

> > Can you shed some lights on why changing `targetBean` to `invocation.getTarget()` can sort it out? The latter can be subclass of the former, is that what you are...

That makes sense for me. What do you think @funky-eyes?

Please provide your use cases and describe problems you encountered in detail.

> @ptyin I would like to work on this issue. Could you please assign it to me? Sure. You can first shed some lights on it (how would you like...