Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[FIX] Clone eqLogic => handle new cmd id #2540

Closed
zoic21 opened this issue Apr 17, 2024 · 2 comments · Fixed by #2727
Closed

[FIX] Clone eqLogic => handle new cmd id #2540

zoic21 opened this issue Apr 17, 2024 · 2 comments · Fixed by #2727
Assignees
@zoic21
Labels
Enhancement
Milestone
4.5

Comments

@zoic21
Copy link
Contributor

zoic21 commented Apr 17, 2024

When we clone eqLogic some information are link to old cmd id (of eqLogic source), need to be replace with new cmd id (from clone result eqLogic) like information field for graph in widget background
@rocket13011 rocket13011 added this to the 4.5 milestone Apr 30, 2024
@rocket13011 rocket13011 changed the title Clone eqLogic => handle new cmd id [FIX] Clone eqLogic => handle new cmd id Apr 30, 2024
@Mips2648 Mips2648 mentioned this issue Jul 2, 2024
Merged
11 tasks
@Mips2648
Copy link
Collaborator

Mips2648 commented Jul 2, 2024

Hello,
Apart backgraph cmdId which I propose to fix in #2727, do you have another case in mind for this issue?

@zoic21
Copy link
Contributor Author

zoic21 commented Jul 3, 2024

Only one I can see it's maybe case like virtual plugin do calcul with cmd. It's a special case sometime it's needed sometime not... So we can close this issue when PR is merge for me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zoic21 zoic21

Labels
Enhancement
Projects
None yet
Milestone
4.5
Development

Successfully merging a pull request may close this issue.

copy eqLogic backGraph::info Mips2648/core
3 participants
@rocket13011 @zoic21 @Mips2648