fix(core): fix configurable operation id decoding when using uuids

[jacobfrantz1]

Fixes issue #2019

Id's were getting JSON stringified when they shouldn't have been. It only worked for integer ids because JSON.stringify(1) == 1. For uuids, JSON.stringify("XXXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXXX") != "XXXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXXX".

Later, when it uses the values in business logic:

vendure/packages/core/src/common/configurable-operation.ts

Lines 459 to 460 in 218561f

	case 'ID':
	return value as any;

, JSON.parse() is not called on ids.

I can't guarantee this doesn't break anything as the change is in a critical area.

[netlify]

✅ Deploy Preview for effervescent-donut-4977b2 canceled.

Name	Link
🔨 Latest commit	6155d4f
🔍 Latest deploy log	https://app.netlify.com/sites/effervescent-donut-4977b2/deploys/6536d96925d32b0007c93f51

[michaelbromley]

I tested this out locally and it solves the issue. The fact that none of the e2e tests are breaking is a good sign that it should be safe.

Thank you for taking the time to investigate and fix this!