Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions website/modules/@apostrophecms/search/views/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ <h1>Search Results</h1>
{% set resultUrl = doc._url %}
{% elif doc.type == 'team-members' %}
{% set meta = 'Team Member' %}
{% set resultUrl = '/meet-the-team' %}
{% set resultUrl = '/cases' %}
{% elif doc.type == 'testimonials' %}
{% set meta = 'Testimonial' %}
{% set resultUrl = '/cases' %}
Expand All @@ -76,7 +76,7 @@ <h1>Search Results</h1>
{% set resultUrl = '/careers' %}
{% elif doc.type == 'business-partner' %}
{% set meta = 'Partner' %}
{% set resultUrl = '/about-us' %}
{% set resultUrl = '/cases' %}
{% else %}
{% set meta = 'Page' %}
{% set resultUrl = doc._url %}
Expand Down
1 change: 1 addition & 0 deletions website/modules/cases-tags/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ module.exports = {
title: 1,
},
shortcut: false,
searchable: false,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎯 Functional Correctness | 🔵 Trivial | ⚡ Quick win

Rebuild or clean the search index when disabling these piece types.

Changing searchable may not remove documents already present in the search index. Verify Apostrophe 4.17.0 behavior and add the required deployment/reindex step.

  • website/modules/cases-tags/index.js#L12-L12: remove existing case-study tag documents.
  • website/modules/categories/index.js#L7-L7: remove existing category documents.
  • website/modules/team-members/index.js#L8-L8: remove existing team-member documents.
📍 Affects 3 files
  • website/modules/cases-tags/index.js#L12-L12 (this comment)
  • website/modules/categories/index.js#L7-L7
  • website/modules/team-members/index.js#L8-L8
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@website/modules/cases-tags/index.js` at line 12, Verify Apostrophe 4.17.0’s
behavior for disabling searchable piece types and add the required
deployment/reindex step to remove stale search documents. Apply this for
case-study tags at website/modules/cases-tags/index.js:12-12, categories at
website/modules/categories/index.js:7-7, and team members at
website/modules/team-members/index.js:8-8.

},
fields: {
add: {
Expand Down
1 change: 1 addition & 0 deletions website/modules/categories/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ module.exports = {
label: 'Category',
pluralLabel: 'Categories',
shortcut: false,
searchable: false,
},
fields: {
add: {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -217,7 +217,7 @@ body {
font-size: 52px;
font-weight: 700;
line-height: 1.22;
letter-spacing: -1px;
letter-spacing: -2px;
max-width: 1180px;
text-wrap: balance;
color: #141414;
Expand Down Expand Up @@ -462,7 +462,7 @@ svg.sicon {
.ca-hero h1 {
font-size: 26px;
line-height: 1.24;
letter-spacing: -0.4px;
letter-spacing: -0.6px;
}

.ca-cue {
Expand Down
1 change: 1 addition & 0 deletions website/modules/team-members/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ module.exports = {
label: 'Team Member',
pluralLabel: 'Team Members',
shortcut: false,
searchable: false,
sort: {
title: 1,
updatedAt: -1,
Expand Down
Loading