FIX:Issue# 847 Remove step not working in tutorial creation

[Matrx123]

Fixes done #847 Remove step not working in tutorial creation

Description

Fixed the issue in which on clicking the remove button selected tutorial step was not deleting.

Related Issue

None

Motivation and Context

This change was required as the tutorial steps were not deleting.
While deleting the selected step the page was reloading due to implementation of react "form" element, due to which page was reloading and execution was not reaching the remove tutorial steps function.

How Has This Been Tested?

I have implemented the change and recreate the Issue described on the Issue description.
i exactly followed what has been described in the original Issue and gone through those steps after implementation.

Screenshots or GIF (In case of UI changes):

Before Fix:
Buggy issue#847.webm

After Fix:
Fix issue#847.webm

Types of changes

• [x ] Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• [ x] My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• [x ] I have added tests to cover my changes.
• [ x] All new and existing tests passed.

[Matrx123]

hi @shivareddy6 i have commited the changes, please review those.
Thanks

[Matrx123]

oh sorry, my bad i must have not removed it in hurry of commiting. thanks.

On Thu, 10 Aug 2023 at 10:43 AM, Shiva Nanda ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In src/components/Tutorials/subComps/RemoveStepModal.jsx <#849 (comment)>: > @@ -6,88 +6,91 @@ import { useDispatch } from "react-redux"; import { removeStep } from "../../../store/actions"; import Snackbar from ***@***.***/material/Snackbar"; import Typography from ***@***.***/material/Typography"; +import Stack from ***@***.***/material/Stack"; Since we are not using this stack anymore, can you remove the import statement. — Reply to this email directly, view it on GitHub <#849 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AIJINRSZIETJBN72S36F3E3XURUWDANCNFSM6AAAAAA3F7SE3M> . You are receiving this because you authored the thread.Message ID: ***@***.***>

-- Thanks Vipin Joshi