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

Added search/filter results for blog posts #890

Merged
merged 19 commits into from
Oct 16, 2024
Merged

Added search/filter results for blog posts #890

merged 19 commits into from
Oct 16, 2024

Conversation

cjyabraham
Copy link
Collaborator

@cjyabraham cjyabraham commented Oct 3, 2024

Relevant issue

PR instance

I've already created the Search/Filter called "Blog" on the live site and synched it with this dev instance. Currently it's set to infinite scroll but not sure if we should use pagination instead. The infinite scroll will need configuring to include the hr element which needs to be wrapped in a div with everything else.

This PR also upgrades plugins.

On Deploy:

  • Save admin options. This will trigger a function to populate the project taxonomy for blog posts by text search. You may need to click it twice if it times out the first time. Once that's done, you can back out this commit since it only needs to be run once.

@thetwopct
Copy link
Collaborator

thetwopct commented Oct 16, 2024

I've integrated the filter and results in to the /blog/ archive

https://pr-890-cncfci.pantheonsite.io/blog/

Without featured/sticky post:

Screenshot-2024-10-16 --17 12 08@2x

With featured/sticky post:

Screenshot-2024-10-16 --17 13 11@2x

There is no need to have the /blog-filter page any more

P.S. I think ACF released a security fix today, so that could likely be updated again

Signed-off-by: James Hunt <[email protected]>
@cjyabraham cjyabraham merged commit 18d9407 into main Oct 16, 2024
5 checks passed
@cjyabraham cjyabraham deleted the blog-filter branch October 16, 2024 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants