Skip to content

feat: enhance relation permissions and accessibility checks#2765

Open
enjeck wants to merge 1 commit into
mainfrom
improve-relation-perms
Open

feat: enhance relation permissions and accessibility checks#2765
enjeck wants to merge 1 commit into
mainfrom
improve-relation-perms

Conversation

@enjeck

@enjeck enjeck commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

This PR has the following improvements:

  • Relation columns can now only link to tables or views you actually have access to, so you can't link to (and expose data from) something you can't see.
  • The target picker now hides tables/views you can't read and the table the column lives in, so you only choose valid targets.
  • If a linked table or view stops being accessible (e.g. it gets unshared), the relation is automatically disabled and its data hidden, instead of leaking stale information.
  • A clear warning now appears on a relation whose target is no longer accessible, telling you to ask for access or pick a different target (right now, the column ID is shown with the warning, but we could just make it empty instead)
Screenshot 2026-06-26 at 06 07 59
  • Newly added or edited relation columns show their linked data immediately, without needing a page refresh.

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not needed
  • 🔙 Backport requests are created or not needed: /backport to stableX.X
  • 📅 Milestone is set
  • 🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@enjeck enjeck requested a review from blizzz as a code owner June 26, 2026 05:06
Signed-off-by: Enjeck C. <patrathewhiz@gmail.com>
@enjeck enjeck force-pushed the improve-relation-perms branch from ce4f34e to b365a3c Compare June 26, 2026 05:06
@blizzz blizzz moved this to 🏗️ In progress in 📝 Productivity team Jun 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: 🏗️ In progress

Development

Successfully merging this pull request may close these issues.

2 participants