-
Notifications
You must be signed in to change notification settings - Fork 132
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
Update "Syntax docs" link in Brim app's query bar drop-down menu #2360
Comments
Thanks @garrisonhess. The new Zed docs site would indeed be a better destination to link to from the app. While doing some research on migrating the docs site to another hosting provider other reasons (brimdata/zed-docs-site#26) I bumped into a Docusaurus limitation that concerned me, but I think I see the fix. We ideally want the app to point to the docs that match the tag of the bundled Zed release, and this is possible today with the docs in GitHub. However, as can be seen in the first table at https://docusaurus.io/docs/versioning, whichever Version is "latest" ends up being served at a URL that lacks the tag. Using the versions on the current Zed docs site as an example, we can find the Zed Thankfully I do think I see a fix if we end up deploying on Netlify, which looks to be the current likely choice. Their redirects config includes a splat option that seems to do the trick in my quick test. So if I put this
I'll leave this comment here as a note-to-self to circle back and test/deploy this config change once the base work to migrate to Netlify is finished. |
Verified in Brim commit ef478a4. In current Brim code (which will soon be GA'ed under the new "Zui" name) the link to the language docs is no longer present at the location shown in the screenshot at the top of this issue. However, we've always linked to it from Help > Zed Syntax Docs in the pull-down menu. As shown in the attached video, that link now goes to the new docs site. Verify.mp4As the notes in #2511 get into it in more detail, since I'm running a Dev release of Brim here that's pointing to a commit hash for a non-GA Zed, here we end up at the "next" version of the Zed docs (i.e., the tip of Zed Thanks again @garrisonhess for pointing this one out to us! |
Hi Brim team,
I was playing with the toy data in Brim yesterday and noticed that the "Syntax docs" link sent me to GitHub, but the new Zed website docs seemed more helpful. This may not be a bug, so please feel free to close this if it's intended! The description below details things.
Describe the bug
Brim app's "Syntax docs" link directs to older GitHub documentation but the new Zed docs are more directly helpful.
Here is the current link: https://github.com/brimdata/zed/blob/v1.1.0/docs/language/README.md
To Reproduce
Open the Brim app and click the query bar's settings drop-down icon.
Expected behavior
Link to: https://zed.brimdata.io/docs/commands/zq/ (or something else on the new Zed website).
Screenshots
![Screen Shot 2022-05-23 at 3 42 00 PM](https://user-images.githubusercontent.com/27266899/169919267-afeaa6b4-a53c-4662-8600-dcc05bb3ff05.png)
Desktop Info
The text was updated successfully, but these errors were encountered: