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.

CommentFileSizeAuthor
#9 Vocabulary description.png18.52 KBvolkswagenchick
#8 recipe-taxonomy-description-d8.patch599 bytesMaddieC
#5 Taxonomy | Umami Food Magazine 2018-08-12 15-28-41.jpg296.07 KBpazhyn
#2 recipe-category-desc-2991563-2.patch609 bytescaptainpants
Taxonomy___Umami_Food_Magazine.png231.43 KBeli-t

Comments

Eli-T created an issue. See original summary.

captainpants’s picture

captainpants commented
StatusFileSize
new recipe-category-desc-2991563-2.patch609 bytes

Here's a go at a decent description.

captainpants’s picture

captainpants commented
Status: Active » Needs review
captainpants’s picture

captainpants commented
Issue tags: +DrupalNorth2018
pazhyn’s picture

pazhyn commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new Taxonomy | Umami Food Magazine 2018-08-12 15-28-41.jpg296.07 KB

Applied patch locally and it looks ok.
Attaching screenshot.

eli-t’s picture

eli-t
he/him
Primary language English
Location Manchester
commented
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

MaddieC commented
Assigned: Unassigned » MaddieC
MaddieC’s picture

MaddieC commented
Assigned: MaddieC » Unassigned
Status: Needs work » Needs review
StatusFileSize
new recipe-taxonomy-description-d8.patch599 bytes
volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new Vocabulary description.png18.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

eli-t
he/him
Primary language English
Location Manchester
commented

RTBC +1

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

lauriii
he/him
Primary language Finnish
Location Finland
commented
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.