Skip to content

get permissions for dependencies in workspace#2057

Merged
kunal0137 merged 16 commits intodevfrom
dependencyPermissionWorkspace
Feb 16, 2026
Merged

get permissions for dependencies in workspace#2057
kunal0137 merged 16 commits intodevfrom
dependencyPermissionWorkspace

Conversation

@tlokeshrao
Copy link
Copy Markdown
Contributor

@tlokeshrao tlokeshrao commented Feb 10, 2026 •
edited
Loading

Description

Goes with SEMOSS/semoss-ui#2686

Changes Made

  • Add tag to dependency output
  • Allow users to edit a workspace when they don't have access to existing dependencies

How to Test

  1. Steps to reproduce/test the behavior
  2. Expected outcomes

Notes

@tlokeshrao tlokeshrao requested a review from a team as a code owner February 10, 2026 19:40
@tlokeshrao tlokeshrao marked this pull request as draft February 10, 2026 19:50
@tevanburen tevanburen linked an issue Feb 11, 2026 that may be closed by this pull request
Update Workspace page MCP tab to show permission level SEMOSS/semoss-ui#2676
Closed
@tevanburen
Copy link
Copy Markdown
Contributor

tevanburen commented Feb 11, 2026

permission should be owner, editor, read only, discoverable, none

@tevanburen tevanburen mentioned this pull request Feb 11, 2026
Merged
@tevanburen
Copy link
Copy Markdown
Contributor

tevanburen commented Feb 12, 2026

  • Need whether or not the user has requested access to each mcp (only for ones marked as discoverable)
  • tag is behaving weirdly, it's returning as a string if 1 but a string array if many. should just always be a string array probably

@tlokeshrao
Copy link
Copy Markdown
Contributor Author

tlokeshrao commented Feb 12, 2026

  • Need whether or not the user has requested access to each mcp (only for ones marked as discoverable)
  • tag is behaving weirdly, it's returning as a string if 1 but a string array if many. should just always be a string array probably

thats how its stored for some reason - if there is one it stores as a single object and if multiple its an array lol

@tevanburen tevanburen marked this pull request as ready for review February 13, 2026 14:27
@kunal0137 kunal0137 merged commit 6475d44 into dev Feb 16, 2026
3 checks passed
@kunal0137 kunal0137 deleted the dependencyPermissionWorkspace branch February 16, 2026 19:34
@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 16, 2026

@CodiumAI-Agent /update_changelog

mattfreshwaters pushed a commit that referenced this pull request Feb 20, 2026
@tlokeshrao @tevanburen @kunal0137
get permissions for dependencies in workspace (#2057)
eb4d3e8 
* fix: get permissions for dependencies in workspace

* fix: pass none

* feat: extra meta

* fix: add tags

* fix: allow users to edit workspace when another exists in the workspace that they do not have access to

* fix: revert changes

* fix: remove from deps when empty

* fix: enabling group concat for multiple db types

---------

Co-authored-by: Van Buren <tevanburen@deloitte.com>
Co-authored-by: Kunal Patel <kunal0137@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Update Workspace page MCP tab to show permission level

3 participants

@tlokeshrao @tevanburen @kunal0137