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.

Sign up for GitHub

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

Jump to bottom

Ensure gaussholder meta data is not indexed #99

Open
1 task
roborourke opened this issue Mar 4, 2021 · 4 comments
Open
1 task

Ensure gaussholder meta data is not indexed #99

roborourke opened this issue Mar 4, 2021 · 4 comments
Labels
should have Should be done, medium priority for now to refine Issues needing refinement.

Comments

@roborourke
Copy link
Contributor

roborourke commented Mar 4, 2021 •
edited
Loading

Steps to reproduce:

  1. Use gaussholder
  2. Add some images
  3. Enable document indexing
  4. Index documents
  5. Guassholder data URIs are in the search index

Acceptance criteria:

  • Filter out all post metadata with the prefix gaussholder_ from search indexing using the ep_post_sync_args_post_prepare_meta on a late priority of 100
@roborourke roborourke added the bug Existing functionality isn't behaving as expected label Mar 4, 2021
@roborourke roborourke added must have Must be done, high priority to refine Issues needing refinement. should have Should be done, medium priority for now and removed bug Existing functionality isn't behaving as expected must have Must be done, high priority labels May 13, 2022
@veselala
Copy link

Hey team! Please add your planning poker estimate with ZenHub @robindevitt @roborourke @shadyvb

@robindevitt
Copy link
Contributor

https://humanmade.com/2016/01/18/lightweight-image-previews-in-wordpress-with-gaussholder/

@veselala
Copy link

@roborourke please share some knowledge around this on our next refinement so that we can discuss the estimation

@roborourke
Copy link
Contributor Author

There's a filter documented here: https://docs.altis-dxp.com/search/indexing/additional-data/

We need to make sure that any post meta keys with the prefix gaussholder_ are not indexed, updated the acceptance criteria accordingly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
should have Should be done, medium priority for now to refine Issues needing refinement.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@roborourke @veselala @robindevitt