Problem/Motivation

#3305745: Add Queue UI integration added support for Queue UI to the 2.x branch, but this plugin is not present in the 3.x branch.

See corresponding commit on the 2.x branch:
https://git.drupalcode.org/project/queue_unique/-/commit/e4efd0558b3cb02...
 

Proposed resolution

Port the plugin to 3.x.
 

CommentFileSizeAuthor
#7 3374940-queue_unique-integrate-with-queue_ui1a.jpg92.14 KBdydave
#7 3374940-7-queue_unique-add-queue_ui-integration.patch791 bytesdydave

Issue fork queue_unique-3374940

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

longwave created an issue. See original summary.

longwave opened merge request !4

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Active » Needs review
a.milkovsky’s picture

a.milkovsky
he/him
Location UTC +2
commented
Status: Needs review » Reviewed & tested by the community

The PR looks good.

eduardo morales alberti’s picture

eduardo morales alberti
Primary language Spanish
Location Spain, 🇪🇺
commented

Tested on a Drupal 10.1.8. RTBC!

DYdave made their first commit to this issue’s fork.

dydave’s picture

dydave commented
StatusFileSize
new 3374940-7-queue_unique-add-queue_ui-integration.patch791 bytes
new 3374940-queue_unique-integrate-with-queue_ui1a.jpg92.14 KB

Hi everyone,

Thank you very much for raising this issue and providing a very helpful patch.

I would like to confirm successfully testing the changes from !MR4 with :

  • Drupal core: 10.2.3
  • Queue UI: 3.1.4
  • Queue Unique : 3.x-dev

 
I would like to confirm that without this patch the Inspect button doesn't display in Queue UI back-end interface, see screenshot below:

Minor code clean-up changes were added at #6, with the resulting patch attached as reference:
3374940-7-queue_unique-add-queue_ui-integration.patch

We would greatly appreciate if a maintainer could please try taking a look at the changes in this ticket, in particular the corresponding merge request !MR4 and give us some comments and feedback.

Feel free to let us know if you have any questions or concerns on any of the changes in the merge request or this ticket in general, we would surely be glad to help.
Thanks in advance !

dydave’s picture

dydave commented
Issue summary: View changes
Related issues: +#3305745: Add Queue UI integration