Skip to content

Conversation

@tomerqodo
Copy link

Benchmark PR appsmithorg#41289

Type: Corrupted (contains bugs)

Original PR Title: chore: Enforce permission while updating instance-config
Original PR Description: ## Description
Slack Thread
EE Counterpart PR: https://github.com/appsmithorg/appsmith-ee/pull/8242

Fixes #Issue Number
or
Fixes Issue URL

Warning

If no issue exists, please create an issue first, and check with the maintainers if the issue is valid.

Automation

/ok-to-test tags="@tag.All"

🔍 Cypress test results

Tip

🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/18408366993
Commit: 698d879
Cypress dashboard.
Tags: @tag.All
Spec:


Fri, 10 Oct 2025 15:02:32 UTC

Communication

Should the DevRel and Marketing teams inform users about this change?

  • Yes
  • No

Summary by CodeRabbit

  • Breaking Changes

    • Config REST endpoints for fetching/updating by name and ACL-guarded config update paths have been removed; clients relying on those endpoints or permissioned fetch/update should adjust.
  • Bug Fixes

    • Simplified config access surface to reduce permission-related complexity and potential inconsistencies.

Original PR URL: appsmithorg#41289

Issues Breakdown

  • Total Issues: 7
  • Easy: 2
  • Medium: 3
  • Hard: 2

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