Skip to content

revert #28 and add tests#29

Merged
sjakobs merged 2 commits into
adjoeio:masterfrom
pm-nilesh-chate:revert-28-PBX-3382-be-address-empty-device-id-incident
Apr 30, 2026
Merged

revert #28 and add tests#29
sjakobs merged 2 commits into
adjoeio:masterfrom
pm-nilesh-chate:revert-28-PBX-3382-be-address-empty-device-id-incident

Conversation

@pm-nilesh-chate
Copy link
Copy Markdown
Contributor

Reverts #28

Issue: All the SaveItemsWithContext fail!
Reason:
This check is being applied to SaveItemsWithContext where hashkey is not required

	if str, ok := key.HashKey().(string); ok && str == "" {
		return ErrInvalidHashKeyValue
	}

sjakobs
sjakobs previously approved these changes Apr 27, 2026
@pm-nilesh-chate pm-nilesh-chate force-pushed the revert-28-PBX-3382-be-address-empty-device-id-incident branch from 909c364 to e6cf924 Compare April 29, 2026 07:08
@pm-nilesh-chate pm-nilesh-chate changed the title Revert "Skipping empty string in update expressions as it used to convert to …" revert #28 and add tests Apr 29, 2026
@pm-nilesh-chate pm-nilesh-chate force-pushed the revert-28-PBX-3382-be-address-empty-device-id-incident branch 2 times, most recently from 2821649 to 36deab2 Compare April 29, 2026 07:16
@pm-nilesh-chate pm-nilesh-chate force-pushed the revert-28-PBX-3382-be-address-empty-device-id-incident branch from 36deab2 to f7501fd Compare April 29, 2026 07:19
@sjakobs sjakobs merged commit b5c55a1 into adjoeio:master Apr 30, 2026
1 check 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.

2 participants