-
Notifications
You must be signed in to change notification settings - Fork 34
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
Passing string value for groupBy
parameter on Partition
class results in Exception
#6403
Comments
groupBy
parameter on Partition metric results in ExceptiongroupBy
parameter on Partition
class results in Exception
@crynobone I have added replication repo for this now https://github.com/benpdavison/nova-issue let me know if any additional steps apart from installing nova etc are needed. |
I do experience this bug as well. A Partition Metric that which used to work fine suddenly produces this error: "Call to a member function getValue() on string"; |
Downgrading to 4.33.0 works around the problem for now. |
@cweiske yep thats what we did |
@cweiske Thank you for the suggestion. Hoping the issue will get fixed soon, thanks in advance to anybody looking into it ;) |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Description:
Exception produced when passing a string param as
groupBy
parameter to Aggregate metric such ascount
function.Call to a member function getValue() on string
onlaravel/nova/src/Metrics/Partition.php(114)
introduced in 4.34.0Detailed steps to reproduce the issue on a fresh Nova installation:
Set up a any Partition metric as per docs https://nova.laravel.com/docs/metrics/defining-metrics.html#partition-metrics
I am using the
count
function within the calculate as per the docs.Ensure 3rd param,
groupBy
is a string, then load up the metric via the dashboard, error should appear via toast.The text was updated successfully, but these errors were encountered: