Delete node on the frontend should remove it from the graph

[AhsanFarooqDev]

Ticket №: #1264

closes #1264

Problem:

Delete node on the frontend should remove it from the graph

As a Proof:

https://www.loom.com/share/a7560e9de6cb4502a5beb2a3af78390e?sid=0dae9776-94fb-4fb1-b367-c67ea076bfd3

[AhsanFarooqDev]

@tomsmith8 @Rassl Kindly review this PR

[Rassl]

Please take a look to my comment

[AhsanFarooqDev]

@Rassl Here, we are simply deleting the selected node from the data store so that it is reflected throughout the entire app. Therefore, the ref_id must match in order to remove the selected node from the graph.

At previous case:

[Rassl]

@Rassl Here, we are simply deleting the selected node from the data store so that it is reflected throughout the entire app. Therefore, the ref_id must match in order to remove the selected node from the graph.

At previous case:

not sure what are you trying to explain here, the only problem in current changes is this lines, for node type topic ref_id is fake, therefore we should get actual id buy finding real node id by name

[AhsanFarooqDev]

not sure what are you trying to explain here, the only problem in current changes is this lines, for node type topic ref_id is fake, therefore we should get actual id buy finding real node id by name

You are right but when the actual id pass to the removeNode function

then at that function the actual ref_id is not match with the fake node type ref_id that is present in data of type NodeExtended
and that node from the graph is not removed means deleting the node is not reflected on graph

[AhsanFarooqDev]

@Rassl Please review the changes now.

You are right to delete a node from backend we need to pass actual id that would be come from topic node

[Rassl]

LGTM