screenshot

"The term name" and "A description of the term." don't add meaningful information, these can be removed.

CommentFileSizeAuthor
#9 taxo-term-descriptions-gone.png30.27 KByoroy
#7 drupal-remove-useless-descriptions-on-taxonomy-term-2847035-7.patch928 bytesgaurav.kapoor
#3 remove-useless-descriptions-term-2847035-2.patch928 bytesjvsouto
#2 remove-useless-descriptions-term-2847035-2.patch928 bytesjvsouto
taxo-term-descriptions.png32.18 KByoroy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

yoroy created an issue. See original summary.

jvsouto’s picture

jvsouto
Primary language Portuguese, Brazil
CreditAttribution: jvsouto commented
FileSize
remove-useless-descriptions-term-2847035-2.patch928 bytes

Patch

jvsouto’s picture

jvsouto
Primary language Portuguese, Brazil
CreditAttribution: jvsouto commented
Status: Active » Needs review
FileSize
remove-useless-descriptions-term-2847035-2.patch928 bytes

Patch

Status: Needs review » Needs work

The last submitted patch, 3: remove-useless-descriptions-term-2847035-2.patch, failed testing.

jvsouto’s picture

jvsouto
Primary language Portuguese, Brazil
CreditAttribution: jvsouto commented
Status: Needs work » Needs review

Oops!

The last submitted patch, 2: remove-useless-descriptions-term-2847035-2.patch, failed testing.

gaurav.kapoor’s picture

gaurav.kapoor CreditAttribution: gaurav.kapoor at OpenSense Labs commented
FileSize
drupal-remove-useless-descriptions-on-taxonomy-term-2847035-7.patch928 bytes

Patch corrected , clear cache before checking whether issue has been fixed or not.

gaurav.kapoor’s picture

gaurav.kapoor CreditAttribution: gaurav.kapoor at OpenSense Labs commented
Status: Needs review » Reviewed & tested by the community
yoroy’s picture

yoroy CreditAttribution: yoroy at Roy Scholten commented
FileSize
taxo-term-descriptions-gone.png30.27 KB

Thank you @jvsouto, that was fast :-)
@gaurav.kapoor, thanks for testing, but I don't see any difference between the last patch @jvsouto posted and yours?

Visual proof via simplytest.me:

RTBC indeed.

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
CreditAttribution: claudiu.cristea as a volunteer and at Webikon commented

@gaurav.kapoor, please stop re-posting patches without changing nothing. I see you did the same with https://www.drupal.org/node/2845644#comment-11888724 and https://www.drupal.org/node/2845483#comment-11888727

gaurav.kapoor’s picture

gaurav.kapoor CreditAttribution: gaurav.kapoor at OpenSense Labs commented

@claudiu.cristea i am new to drupal , i read the correct patch format in the documentation and renamed patches with that format.I wont repost any patches now.

gaurav.kapoor’s picture

gaurav.kapoor CreditAttribution: gaurav.kapoor at OpenSense Labs commented

..

gaurav.kapoor’s picture

gaurav.kapoor CreditAttribution: gaurav.kapoor at OpenSense Labs commented
yoroy’s picture

yoroy CreditAttribution: yoroy at Roy Scholten commented

Thanks for explaining @gaurav.kapoor. It's good to use the preferred naming format for patches but it's not necessary to repost other peoples patches just to change the name.

jvsouto’s picture

jvsouto
Primary language Portuguese, Brazil
CreditAttribution: jvsouto commented

Thank you guys!
Sorry for not writing the title as in the documentation.

  • catch committed 98c5bd3 on 8.3.x 
    Issue #2847035 by jvsouto, yoroy: Remove useless descriptions on...
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.3.x, thanks!

  • catch committed 98c5bd3 on 8.4.x 
    Issue #2847035 by jvsouto, yoroy: Remove useless descriptions on...

  • catch committed 98c5bd3 on 8.4.x 
    Issue #2847035 by jvsouto, yoroy: Remove useless descriptions on...

Status: Fixed » Closed (fixed)

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