Merged
Conversation
Member
andrasbacsai
commented
Mar 19, 2026
- fix service env update returns 404 - uses wrong API endpoint #48
- Changed app and service env update commands to accept only the resource UUID instead of separate env UUID argument - Made --key and --value required flags for identifying and updating environment variables - Removed UUID field from EnvironmentVariableUpdateRequest model - Updated validation to explicitly require --key and --value instead of "at least one field" - Changed ServiceEnvBulkUpdateResponse from struct with message to slice of ServiceEnvironmentVariable - Updated BulkUpdateEnvs return type from pointer to non-pointer - Updated tests and documentation to reflect new command interface
fix: update service env command
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.