Skip to content

refactor: remove deprecation warnings from post response vars in Collection, Folder, and RequestPane settings#6291

Merged
bijin-bruno merged 1 commit intousebruno:release/v2.15.1from
sanjaikumar-bruno:fix/remove-deprecation-warnings-for-post-response
Dec 3, 2025
Merged

refactor: remove deprecation warnings from post response vars in Collection, Folder, and RequestPane settings#6291
bijin-bruno merged 1 commit intousebruno:release/v2.15.1from
sanjaikumar-bruno:fix/remove-deprecation-warnings-for-post-response

Conversation

@sanjaikumar-bruno
Copy link
Member

@sanjaikumar-bruno sanjaikumar-bruno commented Dec 3, 2025 •
edited by coderabbitai bot
Loading

BRU-2259

Summary by CodeRabbit

  • Refactor
    • Removed deprecation warnings from Post Response Variables sections in collection, folder, and request settings for a cleaner user interface.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 3, 2025 •
edited
Loading

Walkthrough

Removes DeprecationWarning imports and JSX usage from Post Response sections across three Vars components (CollectionSettings, FolderSettings, RequestPane). No logic changes—the VarsTable rendering remains unchanged; only the deprecation notice display is eliminated.

Changes

Cohort / File(s) Summary
Remove DeprecationWarning from Post Response sections
packages/bruno-app/src/components/CollectionSettings/Vars/index.js, packages/bruno-app/src/components/FolderSettings/Vars/index.js, packages/bruno-app/src/components/RequestPane/Vars/index.js		 Removed DeprecationWarning import and corresponding JSX usage from Post Response blocks across all three Vars components. No functional changes to state or rendering logic.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Repetitive, homogeneous changes across three files following an identical removal pattern
  • No logic modifications, state changes, or behavioral impact
  • Straightforward cleanup—import removal and JSX deletion

Possibly related PRs

  • PR #6195: Modifies the same Vars components' Post Response UI, making structural changes to response-vars handling alongside this PR's deprecation warning removal.

Suggested labels

size/L

Suggested reviewers

  • helloanoop
  • lohit-bruno
  • bijin-bruno
  • naman-bruno

Poem

A warning bid farewell, thrice removed with care,
Three components, one pattern, deprecated air,
The code breathes lighter now, the clutter swept away,
Simplicity restored—deprecated no more today. 🧹✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: removing deprecation warnings from post response vars across three specific components.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 536b739 and f156047.

📒 Files selected for processing (3)
  • packages/bruno-app/src/components/CollectionSettings/Vars/index.js (0 hunks)
  • packages/bruno-app/src/components/FolderSettings/Vars/index.js (0 hunks)
  • packages/bruno-app/src/components/RequestPane/Vars/index.js (0 hunks)
💤 Files with no reviewable changes (3)
  • packages/bruno-app/src/components/FolderSettings/Vars/index.js
  • packages/bruno-app/src/components/CollectionSettings/Vars/index.js
  • packages/bruno-app/src/components/RequestPane/Vars/index.js

Comment @coderabbitai help to get the list of available commands and usage tips.

@bijin-bruno bijin-bruno merged commit 85b5875 into usebruno:release/v2.15.1 Dec 3, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bijin-bruno bijin-bruno bijin-bruno approved these changes

@helloanoop helloanoop Awaiting requested review from helloanoop helloanoop is a code owner

@lohit-bruno lohit-bruno Awaiting requested review from lohit-bruno lohit-bruno is a code owner

@naman-bruno naman-bruno Awaiting requested review from naman-bruno naman-bruno is a code owner

Assignees

No one assigned

Labels

size/XS

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sanjaikumar-bruno @bijin-bruno