- Status changed to Needs review
almost 2 years ago 3:33pm 24 January 2023 - 🇺🇸United States smustgrave
Rerolled #18
Updated patch to have an access check for the view link
Tweaked the test also. The last submitted patch, 31: 3055600-31.patch, failed testing. View results →
- Status changed to Needs work
almost 2 years ago 12:04pm 25 January 2023 - 🇫🇮Finland lauriii Finland
Are we confident that the view link should be the primary action for the revisions? I'm wondering if it would make sense to at least first add the view button as a secondary action so we don't need to weigh in on that here. We could then change the view to become the primary action later, if we think that would be helpful.
- Status changed to Needs review
almost 2 years ago 4:05pm 4 February 2023 The last submitted patch, 35: 3055600-35.patch, failed testing. View results →
- 🇮🇳India nayana_mvr
Verified the patch #37 and tested it on Drupal version 10.1.x. The patch works fine and I have added the before and after screenshots for reference.
- 🇮🇳India sonam.chaturvedi Pune
Verified and tested patch #37 on 10.1.x-dev.
Patch applied cleanlyTest Result:
"View" link is added as second option to Node revisions overview "Operations" links.
Screenshot already available in previous comment.RTBC+1
- 🇦🇺Australia acbramley
As we now have a generic revision UI in 10.1.x ( ✨ Implement a generic revision UI Fixed ) and Node is going to also use it with 📌 [PP1] Switch Node revision UI to generic UI Needs work why don't we focus on getting node ported across and then enhance the generic UI to add the operation?
- Status changed to Postponed
almost 2 years ago 1:51pm 17 March 2023 - 🇺🇸United States smustgrave
Fine with postponing on 📌 [PP1] Switch Node revision UI to generic UI Needs work