Problem/Motivation

The Recipe Category Taxonomy has no description. We should put some words here to indicate what it is for.

A woefully undescribed taxonomy

Note whilst Ingredients and Recipe Cuisine also have no description, they also have no terms and consensus is for them to be removed under #2990611: Remove empty taxonomies from Umami Demo, so we don't need to add a description for them here.

Proposed resolution

Add a description.

Remaining tasks

  • Decide on description.
  • Add description.

User interface changes

None.

API changes

None.

Data model changes

None.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Eli-T created an issue. See original summary.

captainpants’s picture

Here's a go at a decent description.

captainpants’s picture

Status: Active » Needs review
captainpants’s picture

Issue tags: +DrupalNorth2018
pazhyn’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
296.07 KB

Applied patch locally and it looks ok.
Attaching screenshot.

Eli-T’s picture

Status: Reviewed & tested by the community » Needs work

Thanks for working on this!

I don't think we should add examples this description. The user can easily click list terms to see the actual terms, which serve as better examples. If the nature of the terms changes, we don't have to implement the change in both places (eg if we implemented #2991577: Pluralise taxonomy terms in Umami's Recipe Category vocabulary).

Also I don't think the categories are the best way to serve a recipe, necessarily - they are just what the recipes are for. An accompaniment isn't the best way of serving chili sauce - it's the only way. So maybe at the cost of being less descriptive but more accurate, we could just describe this taxonomy as something like the type of recipe?

MaddieC’s picture

Assigned: Unassigned » MaddieC
MaddieC’s picture

Assigned: MaddieC » Unassigned
Status: Needs work » Needs review
FileSize
599 bytes
volkswagenchick’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
18.52 KB

Patch spun up using the Umami profile on simplytest.me.
I navigated to /admin/structure/taxonomy.
Description is short and sweet and show in the UI. Screenshot attached.

Thanks!! Marking RTBC

Eli-T’s picture

RTBC +1

  • lauriii committed b489195 on 8.7.x
    Issue #2991563 by captainpants, MaddieC, Eli-T, pazhyn, volkswagenchick...
lauriii’s picture

Status: Reviewed & tested by the community » Fixed

Committed b489195 and pushed to 8.7.x. Also cherry-picked to 8.6.x. Thanks!

  • lauriii committed c66ab41 on 8.6.x
    Issue #2991563 by captainpants, MaddieC, Eli-T, pazhyn, volkswagenchick...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.