- Issue created by @ressa
- @ressa opened merge request.
- 🇬🇧United Kingdom schillerm
Hi, I just reviewed and tested this locally on a D10.4.6 site. Everything works as expected and the new render arrays/ page layout make sense. Can't find any problems with spelling / words etc. +1 for RTBC
- 🇩🇰Denmark ressa Copenhagen
Thanks for reviewing @schillerm, feel free to change the Status :)
- 🇩🇰Denmark ressa Copenhagen
Thanks, though it's not clear to me if you tried the patch and evaluated the changes?
I ask because I see that you commented "Patch in #9 works fine, this has been verified by multiple developers, moving this to RTBC." in 🐛 PHP Fatal error: Declaration of Drupal\\flag_anon\\FlagAnonLinkBuilder::build($entity_type_id, $entity_id, $flag_id) Active ...
Changing status to RTBC requires that you apply the patch, and check if the changes make sense:
Reviewed & tested by the community ("RTBC")
[...]
Setting an issue to this status is a judgment call. It means you have thoroughly tested and reviewed the issue [my emphasis] and believe it is ready. If you are unsure, the status should not be changed.From https://www.drupal.org/docs/develop/issues/fields-and-other-parts-of-an-... →