Skip to content

BC-11414 - Refactor Tldraw Server ConfigurationModule to Use Injection Token Register Approach#101

Merged
CeEv merged 8 commits intomainfrom
BC-11414
Mar 12, 2026
Merged

BC-11414 - Refactor Tldraw Server ConfigurationModule to Use Injection Token Register Approach#101
CeEv merged 8 commits intomainfrom
BC-11414

Conversation

@SevenWaysDP
Copy link
Contributor

@SevenWaysDP SevenWaysDP commented Mar 4, 2026

Description

Links to Tickets or other pull requests

Screenshots of UI changes

Approval for review

  • DEV: If the API or client code was changed, all necessary code generation or synchronization steps were completed and tested.
  • QA: In addition to review, the code has been manually tested (if manual testing is possible)
  • All points were discussed with the ticket creator, support-team or product owner. The code upholds all quality guidelines from the PR-template.

- Implemented TypeGuard class with methods for checking types: isError, isNull, isUndefined, isNumber, isString, isArray, etc.
- Added check methods for strict type validation: checkNumber, checkString, checkArray, etc.
- Included methods for object key validation: checkKeyInObject, checkKeysInObject, and checkKeysInInstance.
- Introduced utility methods for retrieving values from objects: getValueFromObjectKey and getValueFromDeepObjectKey.

feat: implement CommaSeparatedStringToArray transformer

- Created CommaSeparatedStringToArray transformer to convert comma-separated strings into arrays of trimmed strings.
- Added tests to validate the transformation behavior for various input scenarios.

feat: add StringToBoolean transformer

- Developed StringToBoolean transformer to convert specific string values ('1', 'true', '0', 'false') to boolean.
- Included error handling for unsupported values.

feat: create StringToNumber transformer

- Implemented StringToNumber transformer to convert string representations of numbers into actual numbers.
- Added tests to ensure correct transformation from string to number.
@CeEv CeEv self-requested a review March 5, 2026 12:31
@CeEv CeEv self-requested a review March 5, 2026 14:00
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 9, 2026

@CeEv CeEv merged commit 1599f4a into main Mar 12, 2026
131 of 133 checks passed
@CeEv CeEv deleted the BC-11414 branch March 12, 2026 11:51
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