Vocabulary listing missing delete operation for vocabularies

Verified the patch #30 and tested it on Drupal version 10.1.x. The patch works fine in 10.1.x also and I have added the before and after screenshots for reference.

Before patch

After patch

This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.

FYI #31 provided screenshots so any following screenshots were duplicative work and not needed.

Testing the code I get the same as everyone else.

Test coverage seems correct and covers the use case.

Looks fine.

Committed cd94d19 and pushed to 10.1.x. Thanks!

Fyi this just caused a random failure in 🐛 Language module shouldn't alter config in sync mode RTBC

Also relooking at the patch I see t() used in the tests. Should we open a follow up to remove?

The order of the taxonomy terms on the page is not deterministic and that's what's causing the random fails. Reverting for that, and so we can also clean up #42.

This should address the random failure and #42.

Perfect!

Committed d2bf0de and pushed to 10.1.x. Thanks!

Automatically closed - issue fixed for 2 weeks with no activity.