- Issue created by @poker10
- Merge request !667Issue #3493916: Recipes are sorted randomly on Recipes tab → (Merged) created by poker10
- 🇸🇰Slovakia poker10
Moving to needs review. I am curious if the test failure is related to this change?
- 🇺🇸United States phenaproxima Massachusetts
Yeah, that failure is probably caused by the change; it's consistent across all runs, and is happening in the test where we specifically use the recipes source. So I guess we gotta fix that, but hey -- at least it's tested!!
- First commit to issue fork.
- 🇺🇸United States phenaproxima Massachusetts
Seems like a reasonable approach but we will need stronger test coverage.
- 🇬🇪Georgia lekso surameli
Thank you for the work
After reviewing the code, I noticed that recipes are currently being sorted by machine_name rather than the human-readable name of the recipe. Sorting by name might provide a more user-friendly experience, as it's generally what end-users would expect to see
In this example bellow you can see how the Website feedback contact form appears before the Full HTML editor, this happens because the machine name of the first recipe is feedback_contact_form
- 🇺🇸United States phenaproxima Massachusetts
Looks great to me. I offered a suggestion to make the test a little more streamlined, but that's truly optional.
- 🇦🇺Australia pameeela
Ohhhh +1, I have been wondering about this but never managed to document it!
-
tim.plunkett →
committed 24be5f9e on 2.0.x authored by
poker10 →
Issue #3493916 by narendrar, poker10, lekso surameli, phenaproxima:...
-
tim.plunkett →
committed 24be5f9e on 2.0.x authored by
poker10 →
- Status changed to Fixed
about 2 months ago 2:59pm 12 February 2025 Automatically closed - issue fixed for 2 weeks with no activity.