Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I am new to Drupal and worked on this module with my mentors, RoySegall and liberatr. We found that with the entity example module's delete links, there was no confirmation message.
I think it would be a great addition to have an options tab along with the delete link that gives you the confirmation to avoid mistakes.
Please check the patch I have attached for errors. This is my first attempt at patching so any comments would be appreciated! Thanks!
-Tierra
Comment | File | Size | Author |
---|---|---|---|
#1 | added_delete_confirmation-0.patch | 2.89 KB | IvoryTierra |
Comments
Comment #0.0
IvoryTierra CreditAttribution: IvoryTierra commentedUpdated issue summary.
Comment #1
IvoryTierra CreditAttribution: IvoryTierra commentedWhoops I forgot the patch.
Comment #3
rfayCongratulations on your first patch! You'll need to run the tests locally and work out the test problem. Improvements to the tests are welcome as well.
Comment #4
Mile23Comment #4.0
Mile23Updated issue summary.
Comment #5
Mile23This patch should properly use confirm_form().
I can understand if the enthusiasm is a little stale. I'm unassigning this, but go ahead and re-assign if you'd like.
Comment #6
jungleNo activity for over 8 years. I am closing this. Please feel free to reopen if necessary.
Thanks!
Comment #7
jungle