Skip to content

feat: add new fields regarding NULL values#1277

Open
fiedlr wants to merge 4 commits intogooddata:masterfrom
fiedlr:afi-cq-1959-add-new-fields-to-pysdk
Open

feat: add new fields regarding NULL values#1277
fiedlr wants to merge 4 commits intogooddata:masterfrom
fiedlr:afi-cq-1959-add-new-fields-to-pysdk

Conversation

@fiedlr
Copy link
Contributor

@fiedlr fiedlr commented Jan 27, 2026

jira: CQ-1959
risk: low

@fiedlr fiedlr changed the title feat: add new fields to pysdk feat: add new fields regarding NULL values Jan 27, 2026
@fiedlr fiedlr force-pushed the afi-cq-1959-add-new-fields-to-pysdk branch 5 times, most recently from df52168 to 34e13fd Compare February 3, 2026 14:03
@fiedlr fiedlr marked this pull request as ready for review February 3, 2026 14:03
@fiedlr fiedlr force-pushed the afi-cq-1959-add-new-fields-to-pysdk branch 6 times, most recently from 98ad039 to 0b1f31f Compare February 3, 2026 16:06
@fiedlr fiedlr marked this pull request as draft February 3, 2026 17:19
@fiedlr fiedlr force-pushed the afi-cq-1959-add-new-fields-to-pysdk branch 8 times, most recently from 2addf6d to 53742b0 Compare February 10, 2026 15:00
@fiedlr fiedlr force-pushed the afi-cq-1959-add-new-fields-to-pysdk branch 3 times, most recently from db8c1ee to c66c70f Compare February 16, 2026 10:13
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needed to de-sync workspace refresh and workspace upload for some tests

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was missing from the localization update (not related to ticket but needed)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We left localization testing in gooddata-python-sdk/packages/gooddata-sdk/tests/catalog/expected/declarative_workspaces.json

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We add this to isolate testing stuff with locales

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should've been in the first commit but we can keep it here

@fiedlr fiedlr force-pushed the afi-cq-1959-add-new-fields-to-pysdk branch from 4e50c4e to 4ca8d2d Compare February 16, 2026 11:50
@fiedlr fiedlr marked this pull request as ready for review February 16, 2026 12:05
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