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

Add Delete Flow #37

Closed
wants to merge 4 commits into from
Closed

Add Delete Flow #37

wants to merge 4 commits into from

Conversation

sumansta
Copy link
Contributor

@sumansta sumansta commented May 12, 2020 •
edited
Loading

  • Add Delete Button on Table Row
  • SweetAlert Confirm Model on button click
  • Delete API on confirm click
  • Remove whole row click from ISSUE#7 Integrate a flow for update #36 and add Edit button for update flow

image
image

Closes #8

@sumansta sumansta self-assigned this May 12, 2020
@sumansta sumansta requested a review from sanjevShakya May 14, 2020 06:20
sanjevShakya
sanjevShakya reviewed May 15, 2020
View reviewed changes
src/components/common/table/Table.js Outdated Show resolved Hide resolved
src/components/home/employees/list/EmployeeList.js Outdated Show resolved Hide resolved
src/components/home/employees/list/EmployeeList.js Outdated Show resolved Hide resolved
src/components/home/employees/list/EmployeeList.js Outdated Show resolved Hide resolved
@sumansta sumansta requested a review from sanjevShakya May 25, 2020 12:09
@sumansta sumansta closed this Mar 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anishmanandhar anishmanandhar Awaiting requested review from anishmanandhar

@pratishshr pratishshr Awaiting requested review from pratishshr

@sanjevShakya sanjevShakya Awaiting requested review from sanjevShakya

Assignees

@sumansta sumansta

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[CRUD] Integrate a flow for "Delete"
2 participants
@sumansta @sanjevShakya