Skip to content

Refactor spaces code#5518

Merged
glpatcern merged 6 commits intomasterfrom
refactor/cleanup-spaceslevel
Mar 2, 2026
Merged

Refactor spaces code#5518
glpatcern merged 6 commits intomasterfrom
refactor/cleanup-spaceslevel

Conversation

@jessegeens
Copy link
Copy Markdown
Contributor

@jessegeens jessegeens commented Feb 23, 2026

This PR:

  • gets rid of the spacesLevel function, which contained hardcoded values for determining at what level a space starts. This is now configurable with the space_depth parameter in the storage provider
  • the space_depth parameter is now also used to determine at what level you can start sharing
  • renamed a number of functions related to the parsing and encoding / decoding of space IDs

@update-docs
Copy link
Copy Markdown

update-docs Bot commented Feb 23, 2026

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@jessegeens jessegeens force-pushed the refactor/cleanup-spaceslevel branch 2 times, most recently from 44f2ea6 to 5953992 Compare February 24, 2026 08:25
@jessegeens jessegeens force-pushed the refactor/cleanup-spaceslevel branch from 5953992 to aa52852 Compare February 24, 2026 10:30
@jessegeens jessegeens force-pushed the refactor/cleanup-spaceslevel branch 2 times, most recently from 74505cd to 31fc491 Compare February 24, 2026 16:19
@jessegeens jessegeens force-pushed the refactor/cleanup-spaceslevel branch 2 times, most recently from 78250ab to ae493e1 Compare February 25, 2026 15:20
@jessegeens jessegeens changed the title prototype for removing spacesLevel function Refactor spaces code Feb 25, 2026
@jessegeens jessegeens marked this pull request as ready for review February 25, 2026 15:30
…nd use space depth to determine when resources can be shared
@jessegeens jessegeens force-pushed the refactor/cleanup-spaceslevel branch from ae493e1 to 93e908c Compare February 25, 2026 15:32
glpatcern
glpatcern previously approved these changes Mar 2, 2026
Comment thread internal/grpc/services/storageprovider/storageprovider.go Outdated
Comment thread pkg/spaces/utils.go
@glpatcern glpatcern merged commit a74c0e9 into master Mar 2, 2026
13 of 15 checks passed
@jessegeens jessegeens deleted the refactor/cleanup-spaceslevel branch April 7, 2026 08:39
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.

2 participants