- 🇬🇧United Kingdom ORION web
@DamienMcKenna @wells is it possible to have some feedback here please?
- Status changed to Needs work
about 2 months ago 3:04pm 19 July 2024 - 🇺🇸United States DamienMcKenna NH, USA
Thank you for putting this together.
With a quick look I think this is the correct approach.
Due to drupalci being shut down, this needs to be turned into a merge request for the 3.0.x branch, then we can confirm whether it still works.
- 🇬🇧United Kingdom ORION web
Thanks for your uber-quick response and good to hear!
Ah yes, I'll prepare a MR :) - First commit to issue fork.
- Merge request !69Issue #3187882 "Description subproperty for organization or person." → (Open) created by ankitv18
- 🇮🇳India ankitv18
@ioanmar I've created a MR on your behalf please do check/validate and update if need.
cc: @DamienMcKenna
- 🇬🇧United Kingdom ORION web
@ankitv18 very speedy indeed but not sure why you ignored my last comment?
- Status changed to Needs review
about 2 months ago 4:46pm 19 July 2024 - 🇮🇳India ankitv18
@ioanmar Oops I'm sorry but your effort isn't in the vein. I saw it's unassigned and thought you would need more time.
Always happy to help!!Btw pipelines are passing for all ~~ hence moving into review.
cc: @DamienMcKenna
- 🇬🇧United Kingdom ORION web
Yes my bad I should’ve assigned it. Thanks for tip (and speed)!
- 🇮🇳India ankitv18
Hi @ioanmar, Could you please validate and review the same changes at your end? And marked this one RTBC so we could take this ahead.
- Status changed to RTBC
about 2 months ago 10:53am 20 July 2024 - 🇺🇸United States DamienMcKenna NH, USA
Thank you both for finishing this off.
I think this needs to go into a 3.1.x release.