Skip to content

fix: remove unused endpoints#7093

Merged
NguyenHoangSon96 merged 1 commit intomasterfrom
fix/remove-unused-api1
Nov 6, 2025
Merged

fix: remove unused endpoints#7093
NguyenHoangSon96 merged 1 commit intomasterfrom
fix/remove-unused-api1

Conversation

@NguyenHoangSon96
Copy link
Contributor

Closes Issue

  • Apis still being called after 2 unused buttons 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 Nov 5, 2025
@NguyenHoangSon96 NguyenHoangSon96 requested review from a team as code owners November 5, 2025 05:45
@NguyenHoangSon96
Copy link
Contributor Author

@wdoconnell @philjb
I have to move this #7081 here because #7081 is from my fork repo, so I can't
"update the PR to merge into master (not influxdata:master)" like @wdoconnell suggested.

@NguyenHoangSon96 NguyenHoangSon96 added this pull request to the merge queue Nov 6, 2025
Merged via the queue into master with commit 0623b4d Nov 6, 2025
6 checks passed
@NguyenHoangSon96 NguyenHoangSon96 deleted the fix/remove-unused-api1 branch November 6, 2025 03:18
@philjb
Copy link
Contributor

philjb commented Nov 6, 2025

@NguyenHoangSon96 -- would you please port this to the release/OSS branch too? 🙇

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.

3 participants