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: delete record in edit header #828

Merged

Conversation

spiritanand
Copy link
Contributor

Resolves #668

UI

Desktop

image

Mobile

image

@spiritanand
Copy link
Contributor Author

Screenshot 2024-08-15 at 10 32 33 PM

@spiritanand spiritanand changed the title [Draft] Feat: Delete record in edit header Feat: Delete record in edit header Aug 15, 2024
@spiritanand
Copy link
Contributor Author

@tea-artist feature has been implemented, authorized users can now delete records from edit menu

@tea-artist tea-artist requested review from Sky-FE and boris-w and removed request for Sky-FE August 16, 2024 03:12
@tea-artist
Copy link
Contributor

Checklist:

  • desktop
  • mobile

  • Grid view
  • Kanban view

  • Shared view

@tea-artist tea-artist changed the title Feat: Delete record in edit header feat: delete record in edit header Aug 16, 2024
@spiritanand
Copy link
Contributor Author

@tea-artist @boris-w users can now delete using the deleteRecord API (only who have permissions record|delete) from all the 3 views in all devices 🫡

@CLAassistant
Copy link

CLAassistant commented Aug 16, 2024

CLA assistant check
All committers have signed the CLA.

@tea-artist tea-artist merged commit e4e1033 into teableio:develop Aug 16, 2024
4 checks passed
@spiritanand spiritanand deleted the feat/delete-record-in-edit-header branch August 16, 2024 12:39
AliceLanniste pushed a commit to AliceLanniste/teable that referenced this pull request Sep 1, 2024
* feat: create ellipsis icon

* feat: ui for dropdown menu and button to delete record

* feat: make delete button uniform

* feat: handle delete of selected record

* refactor: replace ellipsis with more-horizontal icon

* feat: delete record using API

* translation: new key for deleting record in expand edit record
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.

Deleting row from mobile interface
4 participants