Live page docs update #632
Conversation
![hyperlint-ai[bot]](https://avatars.githubusercontent.com/in/718456?s=60&v=4){kind=small}
There was a problem hiding this comment.
The style guide flagged several spelling errors that seemed like false positives. We skipped posting inline suggestions for the following words:
- [Ss]crollable
Note: We resolved prior Hyperlint review comments because:
We updated our inline suggestion AI.
We do this to avoid keeping outdated or irrelevant comments around. We'll leave a new review with current comments below.
Deploying logfire-docs with
|
| Latest commit: |
1903a0d
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://3a933b00.logfire-docs.pages.dev |
| Branch Preview URL: | https://cs-docs-sql-builder.logfire-docs.pages.dev |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #632 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 137 137
Lines 10929 10929
Branches 1524 1524
=========================================
Hits 10929 10929 ☔ View full report in Codecov by Sentry. |
|
|
||
| 6. **Organization selection panel:** Opens a drawer with links to the different organizations you are a member of, and also has links to the Terms and Conditions, Support, Documentation, and a Log Out button. |
There was a problem hiding this comment.
I've deliberately culled a lot of these - some are no longer relevant, but the section is currently just a bit too much of a "wall of text". I've tried to pull out the key/least intuitive areas to describe
There was a problem hiding this comment.
@ChristopherGS I'm going to finish my review and instead just go ahead and commit my changes to your PR to speed this up 👍
There was a problem hiding this comment.
ok @ChristopherGS please ignore my ugly commit history - made some edits to this copy to reflect latest changes, I think we will probably want to improve this more at some point but let's not let perfect be the enemy of better. FYI we should hold off on merging this PR until we make the new SQL editor the default mode - @petyosi could you merge this bad boy once that has happened?
summerscope
changed the title
summerscope
changed the title
Please merge after the new SQL editor is turned on by default @petyosi