-
Notifications
You must be signed in to change notification settings - Fork 188
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
add sql.sequel support #127
Conversation
@modosc, I tried this branch and it did not worked. When I restart server, It generates errors. Had to disable meta_request gem. stderr:
And this is the development.log:
|
@gencer it works with the example repo you built - can you alter that repo so i can reproduce your issue? |
@modosc, Nope. See, If you leave rails monitoring app enabled in development, this things can happen. Monitoring tool I use was enabled at development and probing sequel. When I moved it to production only, it just works. |
what monitoring tool are you referring to? |
Skylight. When I add probe settings and tell that to probe sequel this happens. See here |
this hacks in sequel support by pretending it's
sql.active_record
data.i'm not sure it's the right solution but i'm hesitant to make any ui changes. i thought about altering the tab's title based on some flag but i don't know if people would potentially be using
sequel
andactiverecord
at the same time. for now this probably is a good enough solution?