-
Notifications
You must be signed in to change notification settings - Fork 710
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
[bug 1241661] Calculate CSAT metrics #2788
Conversation
'/surveyresponse?' | ||
'filter[field][0]=datesubmitted' | ||
'&filter[operator][0]=>=&filter[value][0]={start}+0:0:0' | ||
'filter[field][1]=datesubmitted' |
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.
Should this have an '&' at the beginning here?
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.
oops, yes!
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.
funny that it doesn't complain about this when i run the cronjob
This looks fine. That URL is sort of nasty, but I don't think there is any reasonable way to improve it. My above issue about a missing Does this need a new API for the front end? Or do existing APIs cover this already? @brittanystoroz Does this look good to you? |
I was going to put the API stuff in another PR once this landed, but it probably makes sense to just include it here, so that's what I did. ^ |
This looks good to me. Lets do it. r+ |
[bug 1241661] Calculate CSAT metrics
I'm still trying to figure out this campaign thing. But I figured I should push this so at least the front-end work can be unblocked.
r?