Docs for Delete Edit and Test Icons

[K-Kumar-01]

Bug Report 🐛

Click on Delete Icon is not working when the cursor hovers over clause templates.
Package: Contract Editor

Expected Behavior

The template should get deleted.

Current Behavior

There is no deletion of the template.

Steps to Reproduce

1. Hover any template
2. Click on delete icon

Context (Environment)

Desktop

• OS: ZorinOs(Ubuntu based)
• Browser: Chrome
• Version: Chrome- 88.0.4324.182 (Official Build) (64-bit), Zorin OS 15.3

[irmerk]

This is likely an issue with how the icon is used in our Storybook.

[d-e-v-esh]

@irmerk I don't think that the edit button works either. That button is supposed to let us edit the contract, isn't it?

[lokeshn011101]

I am trying to resolve this!

[codetheorem]

Addon action 'delete' is working as it is showing it in the action console.

[d-e-v-esh]

@lokeshn011101 What do you think that the problem is?

[d-e-v-esh]

I can't seem to find where these 3 functions are defined in the code nor the main props of the application. I am not sure if these three functions are defined yet.

The only thing that I could find yet is this.

All this does is log stuff on the actions console.

[dselman]

This is working as designed -- these functions can be supplied by the user of the component. They are not implemented by default because they typically require interaction with other components or the persistence layer of the embedding application.

A nice PR would be to update the README with details like this and how to use the component in general.

[Cronus1007]

@irmerk Shall I work upon this issue.

[Cronus1007]

@irmerk @dselman Before opening the PR the plan that I am intending to follow is :-

• First of all the file that I am going to update is link
• Documenting the role of the user in the Component
• Documenting how to use them in general.

If some sort of addition or deletions are there let me know.

[irmerk]

@Cronus1007 I believe that sounds like a good plan.

[dezfuture]

I would like to contribute and fix the issue.
Can you guide me on how should I go forward into solving this issue.

[irmerk]

@dezfuture as you can see, @Cronus1007 is working on this and has an open PR.

[mansigrover09]

Is this issue resolved?

[mttrbrts]

No, but there is a solution in #305 if you want to open a new PR. Please consider the feedback given in the thread in #305