-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Move GraphiQL menu under Search #12710
Comments
Well technically it is a Query not a Search. |
Agree with @Skrypt, it's a query, we could add a new menu item if it's not there named |
Currently Queries are under Search |
I forgot that, seems that's why Niraj suggested that :) |
Queries are either Search or SQL related this is why they are regrouped together. Wait before anything. Let's discuss about it. |
Search requires query on some data store but I prefer to group search methodologies into a menu item called |
Queries on the other hand also saved into db to execute later |
I agree with @ns8482e. It should not be under configurations. under "Search" is a good place for it since it's just another way to search contents. "Search" from user perspective is not only Lucene or Elastic search. I also think anything configuring the search engines should be moved under |
@Piedone @sebastienros thoughts on this? |
I think the new "Tools" menu as suggested in #17112 makes even more sense than "search". |
Agree. |
Is your feature request related to a problem? Please describe.
Move GraphiQL menu under parent menu Search. It's really a Search and not Configuration.
The text was updated successfully, but these errors were encountered: