Merge pull request databricks-solutions#42 from kishore-mannava-db/main

antonbricks · web-flow · commit 6ef98123ba22 · 2025-09-23T17:05:40.000+02:00
Upload to Volume button is broken in steamlit and dash ui
diff --git a/dash/pages/volumes_upload.py b/dash/pages/volumes_upload.py
@@ -25,7 +25,7 @@ def check_upload_permissions(volume_name: str):
         volume = w.volumes.read(name=volume_name)
         current_user = w.current_user.me()
         grants = w.grants.get_effective(
-            securable_type=SecurableType.VOLUME,
+            securable_type="volume",
             full_name=volume.full_name,
             principal=current_user.user_name,
         )
diff --git a/streamlit/views/volumes_upload.py b/streamlit/views/volumes_upload.py
@@ -22,7 +22,7 @@ def check_upload_permissions(volume_name: str):
         volume = w.volumes.read(name=volume_name)
         current_user = w.current_user.me()
         grants = w.grants.get_effective(
-            securable_type=SecurableType.VOLUME,
+            securable_type="volume",
             full_name=volume.full_name,
             principal=current_user.user_name,
         )

Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,7 @@ def check_upload_permissions(volume_name: str):`
`25`	`25`	`volume = w.volumes.read(name=volume_name)`
`26`	`26`	`current_user = w.current_user.me()`
`27`	`27`	`grants = w.grants.get_effective(`
`28`		`- securable_type=SecurableType.VOLUME,`
	`28`	`+ securable_type="volume",`
`29`	`29`	`full_name=volume.full_name,`
`30`	`30`	`principal=current_user.user_name,`
`31`	`31`	`)`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ def check_upload_permissions(volume_name: str):`
`22`	`22`	`volume = w.volumes.read(name=volume_name)`
`23`	`23`	`current_user = w.current_user.me()`
`24`	`24`	`grants = w.grants.get_effective(`
`25`		`- securable_type=SecurableType.VOLUME,`
	`25`	`+ securable_type="volume",`
`26`	`26`	`full_name=volume.full_name,`
`27`	`27`	`principal=current_user.user_name,`
`28`	`28`	`)`