-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
SQL Lab - A multi-tab SQL editor #514
Conversation
mistercrunch
commented
May 25, 2016
•
edited
edited
ee317ec
to
a90482e
Compare
In which release we will have this feature. |
@naveenh1 we should have a working version of this within a few weeks |
29d2272
to
d0d28e3
Compare
@ascott I think this is ready for a first round of review |
@bkyryliuk the commit log you brought in looks pretty bad, rebase over merge! Ideally there should only have been your commits stacked on top of mine. |
my fault. |
sorry @mistercrunch haven't had a chance to look at this yet, will take a look tonight. |
} | ||
return ( | ||
<Alert bsStyle="info"> | ||
No query history yet... |
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.
we should add a todo that this needs to be wrapper for translation.
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.
Let's just assume all strings exposed to users need to be wrapped somehow, otherwise there'll be todos everywhere...
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.
true enough 👍
* Sync queries using tmp_id. * Fix the unit tests * Bux fixes. Pass 2.
e618f85
to
368706a
Compare
624eee8
to
f861167
Compare