Skip to content

fix: conditionally enable notebook API calls in cloud environments.#7081

Closed
NguyenHoangSon96 wants to merge 1 commit intoinfluxdata:masterfrom
NguyenHoangSon96:fix/remove-unused-api-call
Closed

fix: conditionally enable notebook API calls in cloud environments.#7081
NguyenHoangSon96 wants to merge 1 commit intoinfluxdata:masterfrom
NguyenHoangSon96:fix/remove-unused-api-call

Conversation

@NguyenHoangSon96
Copy link
Contributor

@NguyenHoangSon96 NguyenHoangSon96 commented Oct 14, 2025

Closes Issue

  • Apis still being called after 2 unused button have been removed after Fixing this issue
  • I tried to hard code the CLOUD variable to true, but the UI won't work.
  1. Before the fix.
before_fix.mov
  1. After the fix.
after-fix.mov

Checklist

Authors and Reviewer(s), please verify the following:

  • A PR description, regardless of the triviality of this change, that communicates the value of this PR
  • Well-formatted conventional commit messages that provide context into the change
  • Documentation updated or issue created (provide link to issue/PR)
  • Signed CLA (if not already signed)
  • Feature flagged, if applicable

@NguyenHoangSon96 NguyenHoangSon96 self-assigned this Oct 14, 2025
@NguyenHoangSon96 NguyenHoangSon96 requested review from a team as code owners October 14, 2025 06:31
@NguyenHoangSon96
Copy link
Contributor Author

Hi @philjb
I removed the unused api calls. Please check this PR.
I tried to test in a cloud environment, but I didn't know how to set it up properly to test. 👨‍🔬

@wdoconnell
Copy link
Contributor

@NguyenHoangSon96 Please update the PR to merge into master (not influxdata:master), then amend and re-push to rerun the CI pipeline.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants