- Issue created by @gauravvvv
- 🇧🇩Bangladesh DSushmita Sylhet
Thank you for applying! Reviewers will review the project files, describing what needs to be changed.
Please read Review process for security advisory coverage: What to expect → for more details and Security advisory coverage application checklist → to understand what reviewers look for. Tips for ensuring a smooth review gives some hints for a smoother review.
To reviewers: Please read How to review security advisory coverage applications, What to cover in an application review → , and Drupal.org security advisory coverage application workflow → .
While this application is open, only the user who opened the application can make commits to the project used for the application.
Reviewers only describe what needs to be changed; they don't provide patches to fix what reported in a review. - Status changed to Needs work
over 1 year ago 7:49am 28 August 2023 - 🇮🇳India Harish1688 India
Hi,
Looking the issue schedule transition button size is different for first transition and for second transition, I have added second and third transition but button found the transition button size same as in all states. attached a reference image for this.
can you brief the more details, so i can reproduce it.
Testing Steps:-
1. Drupal Set 10 with claro as the admin theme.
2. For scheduled transitions module follow the same steps mention in the ticket.
3. On node page (node/1/scheduled-transitions), testing the bug.
- 🇧🇩Bangladesh DSushmita Sylhet
Please ignore comment #2.
It was mistakenly put over there. - 🇮🇳India gauravvvv Delhi, India
I have added a screen recording for reference, please see
- Status changed to Active
over 1 year ago 6:32am 29 August 2023 - Status changed to Needs review
over 1 year ago 10:56am 31 August 2023 - last update
over 1 year ago Patch Failed to Apply - 🇮🇳India swatidhurandhar
I was able to reproduce this issue but my button size as well as color was different in first transition. Based on that, I have created a patch to fix it.
- last update
over 1 year ago 29,470 pass - last update
over 1 year ago Patch Failed to Apply - Status changed to Needs work
over 1 year ago 7:16pm 31 August 2023 - 🇺🇸United States smustgrave
11.x is the current development branch.
Is this reproducible with any feature in core? Tagging for steps.
- 🇮🇳India nilesh.k
Hi
This module doesn't support D11. I even tried it with the beta version, but it still gives an error. So, as of now, this module is only for D10.
- First commit to issue fork.
- First commit to issue fork.
- Status changed to Needs review
about 1 year ago 4:06pm 13 November 2023 - 🇮🇳India ameymudras
Was able to reproduce the issue on D11 as well following the steps provided above. I have created a MR with the patch to fix
- Status changed to Needs work
about 1 year ago 4:27pm 13 November 2023 The Needs Review Queue Bot → tested this issue.
While you are making the above changes, we recommend that you convert this patch to a merge request → . Merge requests are preferred over patches. Be sure to hide the old patch files as well. (Converting an issue to a merge request without other contributions to the issue will not receive credit.)
I tried to replicate the issue but for me, the schedule button is looking good. Not able to replicate the issue.
Tested on Drupal 10
Thanks- Status changed to Needs review
about 1 year ago 3:22pm 14 December 2023 - Status changed to Postponed: needs info
about 1 year ago 3:38pm 14 December 2023 - 🇺🇸United States smustgrave
Should he put in this status as there’s nothing to review. If someone can verify the issue, using just core. If not this will be closed
- 🇳🇿New Zealand quietone
The issue summary says that a contrib module needs to be installed? Is this reproducible from core only?
In any case, this issue has been waiting for 10 months for someone to verify that the problem still exists after testing in #16 was not able to reproduce the problem. Therefore, it is time to close this.