Skip to content

feat(gooddata-sdk): [AUTO] add excludeUserSettings parameter to workspace settings resolution methods#1331

Open
yenkins-admin wants to merge 1 commit intomasterfrom
feature/auto-P016-fiscal-calendar-settings
Open

feat(gooddata-sdk): [AUTO] add excludeUserSettings parameter to workspace settings resolution methods#1331
yenkins-admin wants to merge 1 commit intomasterfrom
feature/auto-P016-fiscal-calendar-settings

Conversation

@yenkins-admin
Copy link
Contributor

Added exclude_user_settings parameter to workspace settings resolution SDK methods to support excluding user-level settings from resolution.

Problem: P016
Workflow: https://github.com/gooddata/gdc-nas/actions/runs/22094366952

Changes in gdc-nas

Added excludeUserSettings query parameter to all 4 settings resolution endpoints in metadata-api. This boolean parameter (default: false) allows excluding user-level settings from resolution.

Commits:

  • 8ce1041: Merge pull request #20044 from Vojtasii/vto/fiscal
  • a674e74: feat: add excludeUserSettings support to setting resolution

SDK Implementation

Added exclude_user_settings parameter (default: False) to the following methods in CatalogWorkspaceService:

  • resolve_all_workspace_settings() - now accepts exclude_user_settings and passes it to workspace_resolve_all_settings()
  • resolve_workspace_settings() - now accepts exclude_user_settings and passes it to workspace_resolve_settings()

This enables SDK users to exclude user-level settings from resolution when the parameter is set to True.

Files Modified:

  • packages/gooddata-sdk/src/gooddata_sdk/catalog/workspace/service.py

JIRA: TBD (Jira sandbox currently in maintenance)
Risk: Low

…pace settings resolution methods

Added exclude_user_settings parameter (default: False) to resolve_all_workspace_settings()
and resolve_workspace_settings() methods in CatalogWorkspaceService to enable SDK users
to exclude user-level settings from resolution, matching the API client capability.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
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.

1 participant