-
Notifications
You must be signed in to change notification settings - Fork 606
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
Can not view results-index - MiniProfiler.list is undefined #80
Comments
"+1" doesn't work in the latest version, or in v.3.0.10. |
+1. same issue. can't use the list page. i think the issue is that the js is looking for a div with class profiler-result-full but the controller is returning an empty body. |
+1. Maybe http://stackoverflow.com/a/25776123/593779 gives some clues (haven't checked them out myself). |
👍 Still an issue on latest nuget package |
Chrome console shows on this page:
and the line 5 is
It has something to do with VS Browser Link (or not). After disabling it the error change:
|
+1 |
1 similar comment
+1 |
any feedback on this? |
+1 |
i++ |
bueller... bueller....... .. bueller..... |
To all who are waiting on this: this feature was built by someone who used to be (but is no longer) active on the project. WIth the upgrade to v3 a while back, something about the refactoring broke this. I spent some time trying to figure out why it isn't working. As you can tell by the lack of progress on this issue, I was not successful. And unfortunately, since this is not something that we really use anymore, I cannot justify spending more time on this right now. So from my perspective, unfortunately, this is an abandoned feature that will not come back... ...however, if someone wants to spend some time on this, get it working and submit the needed changes in a Pull Request, I would be ore than happy to merge that in. |
+1 ;( I'm gonna see if I can't fix it |
+1 digitalpacman - you will be my hero if you can fix this... |
+1 same problem and in the page source I have line 5: <script type='text/javascript'>MiniProfiler.list.init({path: '/cwa/mini-profiler-resources/', version: 'uZxmJ8Mst1yfwoTlTBen3ZVQs12agJzT0AK6jmyml/s='})</script> |
+1 |
I've just committed the fix for this repository and ui repository. Hope it helps... |
Nice, when does the nuget package update? |
Thank you :) It didn't take so much time exactly because I copied it from the v2. I could say it took nearly one hour... |
Fix issue #80: Can not view results-index
I downloaded the project and built the package, I uploaded the nupkg to a MyGet feed so I can use it in my project, I'm not sure if I'm allowed to share this link (regarding license rights), so if this is ok, I can share it here until we have an official NuGet package updated |
This was just included in nuget 3.2.0.157 |
Feature to access the last 100 profiled requests via ~/mini-profiler-resources/results-index results in a javascript error that MiniProfiler.list is undefined.
Note that on application start Results_List_Authorize is set:
MiniProfiler.Settings.Results_List_Authorize = r => true;
The text was updated successfully, but these errors were encountered: