Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PLANET-6652: Extract search filters #1691

Merged
merged 5 commits into from
Jun 14, 2022
Merged

PLANET-6652: Extract search filters #1691

merged 5 commits into from
Jun 14, 2022

Conversation

lithrel
Copy link
Contributor

@lithrel lithrel commented Apr 27, 2022

Ref: https://jira.greenpeace.org/browse/PLANET-6652
Ref: https://jira.greenpeace.org/browse/PLANET-6797
Ref/ https://jira.greenpeace.org/browse/PLANET-6578

Clarify search code by matching native taxonomy with class structure

  • Extract search filters to their own classes
  • Rename page_types to post_types
  • Hide tags search
  • Add p4_action type to search

@lithrel lithrel force-pushed the fix/search-taxonomy-6552 branch from 8d38edc to 0a1d1a6 Compare April 27, 2022 07:49
src/Search.php Outdated Show resolved Hide resolved
src/Search/Filters/Tags.php Outdated Show resolved Hide resolved
Base automatically changed from fix/search-taxonomy-6552 to master April 27, 2022 11:19
Extract search filters to their own classes
Use consistent taxonomy
@lithrel lithrel force-pushed the search-extract-filters branch from 0f932be to 9bd771d Compare May 13, 2022 09:25
lithrel added 2 commits May 13, 2022 10:38
Remove tags in search template
Add p4_action type to search filters and template
@lithrel lithrel force-pushed the search-extract-filters branch from 8d989da to 18b2329 Compare May 13, 2022 10:03
@lithrel lithrel changed the title Search: extract search filters PLANET-6652: Extract search filters May 13, 2022
planet-4 added a commit to greenpeace/planet4-test-phobos that referenced this pull request May 25, 2022
/unhold 46cfbfc6-0437-4b01-af59-7b39c07aa78d
planet-4 added a commit to greenpeace/planet4-test-phobos that referenced this pull request Jun 8, 2022
/unhold 5c520759-88b1-455f-a12f-506851a41ceb
@planet-4
Copy link
Contributor

planet-4 commented Jun 8, 2022

Test instance is ready 🚀

🌑 phobos | admin | blocks report | CircleCI | composer-local.json

⌚ 2022.06.13 10:25:08

planet-4 added a commit to greenpeace/planet4-test-phobos that referenced this pull request Jun 9, 2022
/unhold 2a441b74-6fe5-4e27-a7b1-644c71c4cd13
Remove act pages filter and various exceptions
@lithrel lithrel force-pushed the search-extract-filters branch from 7f58009 to 6b26ca4 Compare June 9, 2022 12:49
planet-4 added a commit to greenpeace/planet4-test-phobos that referenced this pull request Jun 9, 2022
/unhold 0caa8465-dab6-4521-b0cf-692e4ec63387
@comzeradd
Copy link
Member

@lithrel is Actions showing up for you locally? I tried various things but couldn't make it work.

I replicated my steps on the test instance.

  1. Searched for "climate" and found one page that was showing up.
  2. Converted it to an Action with the Post Switcher plugin.
  3. Re-sync ES
  4. Searched again for "climate".
    Result: the page no longer shows up in results and no "Actions" filter is visible on the sidebar

Missing action post type in type list
Reduce label definition code
planet-4 added a commit to greenpeace/planet4-test-phobos that referenced this pull request Jun 13, 2022
/unhold bc98010f-ed44-42dd-a561-2e3448b3c99d
@lithrel lithrel self-assigned this Jun 13, 2022
@lithrel
Copy link
Contributor Author

lithrel commented Jun 13, 2022

@comzeradd right, 👍 the post type was missing for the general search args, fixed now 🔧 the "Action" content type appears for your page
You will see the "Action types" section if you create and assign an Action type

Copy link
Member

@comzeradd comzeradd left a comment

Choose a reason for hiding this comment

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

Code looks good and tested locally again 👍

@lithrel lithrel merged commit 1f17a38 into master Jun 14, 2022
@lithrel lithrel deleted the search-extract-filters branch June 14, 2022 15:03
@lithrel lithrel removed the Review label Jun 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants