-
Notifications
You must be signed in to change notification settings - Fork 1
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
HS-968801 - Fix note filter #777
Conversation
I think the typescript check is failing because the API changes aren't yet in production. |
This pull request is automatically being deployed by Amplify Hosting (learn more). |
985d281
to
626bbb6
Compare
8853604
to
611283c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have reviewed the code, made some changes, but it looks great. I'm going to merge this into staging to ensure it's working with Shelby's new API changes.
Does it work ok on the PGA report? |
Good catch. It's no longer producing an error, but it doesn't filter the contacts on |
Seems like a bigger issue. A few filters on PGA report don't work. I'm looking into fixing this on this PR. |
… fixing Search notes filters
I imagine this will need some more work, but it is what I could get figured out today.
Helpscout
The search notes filter is currently throwing errors, so Shelby made a fix to the API which changes the shape of the GraphQL query. I have tested this on the contacts page locally, both as a saved filter and not-saved filter. The changes to
FilterPanel
lines 640-642 were purely for the test's benefits, this code was not reached during my testing on the site. It makes me curious if there are some improper assumptions somewhere, or if there is another way to reach that code.