Skip to content

Conversation

@suthar26
Copy link
Contributor

@suthar26 suthar26 commented Aug 8, 2025

  • added variable metadata
  • updated metadata and context fields to be a dataclass and json outputs
  • update django app to use the local version of the sdk
  • tset using the django app and hooks

@suthar26 suthar26 requested a review from a team as a code owner August 8, 2025 18:49
sdk_variable.ParseFromString(var_bytes)

return pb_utils.create_variable(sdk_variable, default_value)
return pb_utils.create_variable(sdk_variable, default_value), sdk_variable._feature.value
Copy link
Contributor Author

Choose a reason for hiding this comment

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

since we return the variable here, i am adding the featureId as a return here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

and Nullable string has .value property on it, if i dont add this the structure of feature_id is { value : Id }

@suthar26 suthar26 enabled auto-merge (squash) August 8, 2025 20:05
@suthar26 suthar26 merged commit a66ac4f into main Aug 8, 2025
12 checks passed
@suthar26 suthar26 deleted the parthsuthar/cor-3514-add-_feature-as-a-property-in-python-variable-details-in-the branch August 8, 2025 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants