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.

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

feat(v2): implement admin form delete logic feature #3612

Merged
merged 6 commits into from
Mar 29, 2022

Conversation

karrui
Copy link
Contributor

@karrui karrui commented Mar 18, 2022

Problem

This PR implements the delete logic feature.

Tracked by #3365

Solution

Features:

  • feat: add and open DeleteLogicModal on delete icon button click
  • feat: implement delete logic feature

Before & After Screenshots

msw mock for deleting of logic has been added. Should be able to test deletion of logic.

@karrui karrui requested review from mantariksh and tshuli March 18, 2022 09:57
Base automatically changed from form-v2/logic/logic-redesign-1.0 to form-v2/develop March 24, 2022 03:22
@karrui karrui force-pushed the form-v2/logic/delete-logic-modal-2.0 branch from 122bdf2 to cc71f2d Compare March 24, 2022 03:23
Copy link
Contributor

@tshuli tshuli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, except only the toast seems off alignment (or too wide?)

Screenshot 2022-03-28 at 3 05 03 PM

@karrui
Copy link
Contributor Author

karrui commented Mar 29, 2022

lgtm, except only the toast seems off alignment (or too wide?)

Yea, known issue with ChakraUI which I actually fixed upstream (chakra-ui/chakra-ui#4922) (via @seaerchin), but haven't back-ported it back into this repository.

@karrui karrui merged commit b9c9408 into form-v2/develop Mar 29, 2022
@karrui karrui deleted the form-v2/logic/delete-logic-modal-2.0 branch March 29, 2022 04:18
@justynoh justynoh mentioned this pull request Oct 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants