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

Better error messaging for failed requests #28

Closed
mckdemps opened this issue Feb 26, 2021 · 2 comments · Fixed by #58
Closed

Better error messaging for failed requests #28

mckdemps opened this issue Feb 26, 2021 · 2 comments · Fixed by #58
Assignees
Milestone

Comments

@mckdemps
Copy link

Is your enhancement related to a problem? Please describe.

Describe the solution you'd like

We need to interpret log files for failed requests and instruct the user what meta data needs to be added to the index.

Designs

Describe alternatives you've considered

Additional context

@felipeelia
Copy link
Member

Some initial scenarios we can capture:

  • post_date_gmt being a text field => It indicates a sync is needed (deleting the index and saving a post should trigger this error)
  • sorting on a field that is not indexed

We can start with an array indexed by a regular expression and valued by the message to be displayed. The regex should capture important info and feed it to the message (if it failed due to a meta field, that field should be present in the message)

@felipeelia felipeelia added this to the 2.2.0 milestone Oct 25, 2022
@felipeelia felipeelia assigned MARQAS and unassigned felipeelia Oct 25, 2022
@MARQAS MARQAS mentioned this issue Nov 15, 2022
4 tasks
@felipeelia
Copy link
Member

This should now rely on the functionality built in the main plugin (here). Probably some change will be needed there, so we can reuse it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants