Skip to content
This repository was archived by the owner on Feb 16, 2023. It is now read-only.

Commit b91a7e9

Browse files
committed
Fix field count
If an item in the plan had 2 fields in it, the migration apply would still count only one. Basically it would count the number of items instead of number of fields. Now it's fixed
1 parent 004c4b8 commit b91a7e9

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

internals/secrethub/migrate.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -522,14 +522,14 @@ func (cmd *MigrateApplyCommand) Run() error {
522522
return err
523523
}
524524
template.AddField(field.Name, value, field.Concealed)
525+
createCount++
525526
}
526527

527528
changes = append(changes, itemCreation{
528529
vault: vault.Name,
529530
item: item.Name,
530531
itemTemplate: template,
531532
})
532-
createCount++
533533
} else {
534534
opFields, err := onepassword.GetFields(vault.Name, item.Name)
535535
if err != nil {

0 commit comments

Comments
 (0)