feat(ui): python onboarding updating to influxdb3 python #6770
Conversation
mavarius
changed the title
There was a problem hiding this comment.
@mavarius Good work! 👍 I trust you on all the codeSnippet since I don't have the context for the content of the tutorial. Everything else in code looks good to me except one bug:
Seems like there is a bug on the step 7 "Execute an Aggregate Query" when the feature flag ioxOnboarding is set to true. See video below:
Screen.Recording.2023-07-21.at.4.55.48.PM.mov
| <WriteDataDetailsContextProvider> | ||
| {this.renderStep()} | ||
| {isFlagEnabled('ioxOnboarding') | ||
| ? this.renderSqlStep() | ||
| : this.renderStep()} | ||
| </WriteDataDetailsContextProvider> |
There was a problem hiding this comment.
Nice! This flow is cleaner than before 👍
|
You have to refresh the page if you switch the flag. The Serverless version of the Java instructions doesn't have the aggregate step at all.
|
There was a problem hiding this comment.
@mavarius My bad! I was looking at the Java onboarding flow instead of the Python onboarding flow 🤦♀️ I double checked and everything flows nicely on my local machine. Approving now. Sorry for the delay!
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
Closes #6761
This PR updates the Python onboarding with the new client library and adds InfluxQL aggregation instructions.
Checklist
Authors and Reviewer(s), please verify the following: