-
Notifications
You must be signed in to change notification settings - Fork 44
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#547 Add new fields into Model entity #577
Conversation
6c7b66f
to
3bbf4b4
Compare
int32 enhancedSetupFlowOptions = 21 [(gogoproto.moretags) = "validate:\"omitempty,gte=0,lte=65535\""]; | ||
string enhancedSetupFlowTCUrl = 22 [(gogoproto.moretags) = "validate:\"required_if=EnhancedSetupFlowOptions 0,omitempty,url,startsnotwith=http:,max=256\""]; | ||
int32 enhancedSetupFlowTCRevision = 23 [(gogoproto.moretags) = "validate:\"required_if=EnhancedSetupFlowOptions 0,gte=0,lte=65535\""]; | ||
string enhancedSetupFlowTCDigest = 24 [(gogoproto.moretags) = "validate:\"required_if=EnhancedSetupFlowOptions 0,omitempty\""]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to the documentation, the maximum length of this field is 128.
model.EnhancedSetupFlowTCUrl = msg.EnhancedSetupFlowTCUrl | ||
model.EnhancedSetupFlowTCRevision = msg.EnhancedSetupFlowTCRevision | ||
model.EnhancedSetupFlowTCDigest = msg.EnhancedSetupFlowTCDigest | ||
model.EnhancedSetupFlowTCFileSize = msg.EnhancedSetupFlowTCFileSize |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The EnhancedSetupFlowTCFileSize
field can only be set if EnhancedSetupFlowOptions
is present.
model.EnhancedSetupFlowTCDigest = msg.EnhancedSetupFlowTCDigest | ||
model.EnhancedSetupFlowTCFileSize = msg.EnhancedSetupFlowTCFileSize | ||
model.MaintenanceUrl = msg.MaintenanceUrl | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we should reset these fields when EnhancedSetupFlowOptions is set?
3bbf4b4
to
c7aaa1f
Compare
Signed-off-by: Abdulbois <[email protected]> Signed-off-by: Abdulbois <[email protected]>
c7aaa1f
to
f9492ab
Compare
No description provided.