Skip to content

Conversation

@vid277
Copy link
Contributor

@vid277 vid277 commented May 8, 2025

Please indicate what issue this PR is related to and @ any maintainers who are relevant

Example Request:

curl --request POST \
  --url http://localhost:8090/api/organization/update_dataset_configs \
  --header 'Authorization: tr-******************************' \
  --header 'Content-Type: application/json' \
  --header 'TR-Organization: ******************************' \
  --data '{
  "organization_id": "******************************",
  "from_configuration": {
    "LLM_DEFAULT_MODEL": "gpt-4o-mini"
  },
  "to_configuration": {
    "LLM_DEFAULT_MODEL": "gpt-4o",
    "SYSTEM_PROMPT": "You are a helpful assistant"
  }
}'

@vid277 vid277 force-pushed the vs/refactor-update-dataset-configs-with-match branch from b6d0965 to 7fc0a9e Compare May 8, 2025 06:20
}))]
pub struct UpdateAllOrgDatasetConfigsReqPayload {
/// The configuration to provide a filter on what datasets to update.
pub from_configuration: Option<serde_json::Value>,
Copy link
Contributor

Choose a reason for hiding this comment

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

change this to match_configuration

@vid277 vid277 force-pushed the vs/refactor-update-dataset-configs-with-match branch from 504ce41 to 309af0c Compare May 15, 2025 17:16
@cdxker cdxker force-pushed the vs/refactor-update-dataset-configs-with-match branch from 164bb4b to b98c8b3 Compare May 20, 2025 17:13
@cdxker cdxker merged commit a6844cf into main May 21, 2025
3 checks passed
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.

refactor/bugfix: modify update all dataset configurations to be a match on an existing type

4 participants