Problem/Motivation

There is a small typo that I noticed in the UI when creating a new facet.

The raw value of a result item is used to identify a resuls item. It is up to you to ensure that the combination of the result sets makes sense. As the combination bases on the raw calues it makes sense to place this processor on an early position, especially before the URL handler...

Steps to reproduce

Create a new facet and read the UI instructions near "Combine Facets"

Proposed resolution

Update the code to fix the typos.

Note: This issue was created as a novice issue and should be used to help new contributors create patches/PRs.

CommentFileSizeAuthor
#8 Screenshot 2022-11-21 at 10.34.40 AM.png854.73 KBArti Anil Pattewar
#4 3320556-4.patch1.2 KBroshni_k

Comments

dorficus created an issue. See original summary.

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented

Just to amplify:

Note: This issue was created as a novice issue and should be used to help new contributors create patches/PRs.

roshni_k’s picture

roshni_k commented
Assigned: Unassigned » roshni_k
roshni_k’s picture

roshni_k commented
Status: Active » Needs review
StatusFileSize
new 3320556-4.patch1.2 KB

Updated the description

roshni_k’s picture

roshni_k commented
Assigned: roshni_k » Unassigned
prodigyjoe’s picture

prodigyjoe commented
Assigned: Unassigned » prodigyjoe
prodigyjoe’s picture

prodigyjoe commented

Will be working on this this weekend

Arti Anil Pattewar’s picture

Arti Anil Pattewar commented
StatusFileSize
new Screenshot 2022-11-21 at 10.34.40 AM.png854.73 KB

Patch #4 applied successfully on D9.4.x. Misspelled word in description got resolved.

Arti Anil Pattewar’s picture

Arti Anil Pattewar commented
Assigned: prodigyjoe » Unassigned
Status: Needs review » Reviewed & tested by the community

  • mkalkbrenner committed 8d903c7 on 3.0.x 
    Issue #3320556 by Roshni_Kodiganti: Misspelled word in description

  • mkalkbrenner committed ff44940 on 2.0.x 
    Issue #3320556 by Roshni_Kodiganti: Misspelled word in description
mkalkbrenner’s picture

mkalkbrenner
Primary language German
Location 🇩🇪
commented
Version: 2.0.x-dev » 3.0.x-dev
Status: Reviewed & tested by the community » Fixed
volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented

@Arti Anil Pattewar There is no need to attach a screenshot of the terminal where the screenshot applied.
It is better to describe the changes that happen in consideration of the expectations.
Thanks.

https://www.drupal.org/docs/develop/issues/issue-procedures-and-etiquett...
# 13 - "Don't: Upload screenshots of a patch applying. This is of no benefit to maintainers or other contributors."

Status: Fixed » Closed (fixed)

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