updated document schemas #2313

Omri-Levy · 2024-04-22T07:50:28Z

User description

Description

Elaborate on the subject, motivation, and context.

Related issues

Provide a link to each related issue.

Breaking changes

Describe the breaking changes that this pull request introduces.

How these changes were tested

Describe the tests that you ran to verify your changes, including devices, operating systems, browsers and versions.

Examples and references

Links, screenshots, and other resources related to this change.

Checklist

[] I have read the contribution guidelines of this project
[] I have read the style guidelines of this project
[] I have performed a self-review of my own code
[] I have commented my code, particularly in hard-to-understand areas
[] I have made corresponding changes to the documentation
[] My changes generate no new warnings and errors
[] New and existing tests pass locally with my changes

Type

enhancement, bug_fix

Description

Enhanced document schemas for Ghana and Uganda, adding new categories and refining existing ones.
Updated various packages to new versions, ensuring compatibility and incorporating the latest changes in document schemas.
Updated changelogs and package.json files across multiple projects to reflect new versions and dependency updates.

Changes walkthrough

Relevant files

Enhancement

2 files

GH.ts `Enhance and Add New Document Schemas for Ghana` packages/common/src/schemas/documents/workflow/documents/schemas/GH.ts Added new document schemas for financial information and proof of registration in Ghana. Enhanced existing schemas with additional properties and validation rules.	+131/-0
UG.ts `Introduce New Document Schemas for Uganda` packages/common/src/schemas/documents/workflow/documents/schemas/UG.ts Introduced new document schemas for proof of registration and ownership in Uganda. Updated trade license and association letter schemas with additional details.	+147/-0

Documentation

9 files

CHANGELOG.md `Update Changelog for Backoffice-v2` apps/backoffice-v2/CHANGELOG.md Updated changelog to reflect new patch version 0.7.6 with updated dependencies.	+9/-0
CHANGELOG.md `Update Changelog for KYB App` apps/kyb-app/CHANGELOG.md Updated changelog to include new patch version 0.3.7 with dependency updates.	+8/-0
CHANGELOG.md `Update Changelog for Headless Example` examples/headless-example/CHANGELOG.md Updated changelog to reflect new patch version 0.3.6 with updated dependencies.	+8/-0
CHANGELOG.md `Update Changelog for Common Package` packages/common/CHANGELOG.md Added entry for version 0.9.4 detailing updated document schemas.	+6/-0
CHANGELOG.md `Update Changelog for Workflow Core` packages/workflow-core/CHANGELOG.md Updated changelog to include new patch version 0.6.7 with dependency updates.	+7/-0
CHANGELOG.md `Update Changelog for Web UI SDK` sdks/web-ui-sdk/CHANGELOG.md Updated changelog to include new patch version 1.5.5 with dependency updates.	+7/-0
CHANGELOG.md `Update Changelog for Workflow Browser SDK` sdks/workflow-browser-sdk/CHANGELOG.md Updated changelog to include new patch version 0.6.7 with dependency updates.	+8/-0
CHANGELOG.md `Update Changelog for Workflow Node SDK` sdks/workflow-node-sdk/CHANGELOG.md Updated changelog to include new patch version 0.6.7.	+6/-0
CHANGELOG.md `Update Changelog for Workflows Service` services/workflows-service/CHANGELOG.md Updated changelog to include new patch version 0.7.6 with dependency updates.	+9/-0

Configuration changes

11 files

package.json `Update Package Version and Dependencies for Backoffice-v2` apps/backoffice-v2/package.json Updated version to 0.7.6. Updated dependencies to include new versions of common and SDK packages.	+4/-4
package.json `Update Package Version and Dependencies for KYB App` apps/kyb-app/package.json Updated version to 0.3.7. Updated dependencies to newer versions of common and SDK packages.	+3/-3
package.json `Update Package Version and Dependencies for Headless Example` examples/headless-example/package.json Updated version to 0.3.6. Updated dependencies to include new versions of common and SDK packages.	+3/-3
package.json `Update Version for Common Package` packages/common/package.json Updated version to 0.9.4.	+1/-1
package.json `Update Package Version and Dependencies for Workflow Core` packages/workflow-core/package.json Updated version to 0.6.7. Updated dependency to the latest version of the common package.	+2/-2
pnpm-lock.yaml `Update PNPM Lock File for Dependency Management` pnpm-lock.yaml Updated versions and dependencies across various packages to ensure consistency.	+16/-16
package.json `Update Package Version and Dependencies for Web UI SDK` sdks/web-ui-sdk/package.json Updated version to 1.5.5. Updated dependency to the latest version of the common package.	+2/-2
package.json `Update Package Version and Dependencies for Workflow Browser SDK` sdks/workflow-browser-sdk/package.json Updated version to 0.6.7. Updated dependencies to include new versions of common and core packages.	+3/-3
package.json `Update Package Version and Dependencies for Workflow Node SDK` sdks/workflow-node-sdk/package.json Updated version to 0.6.7. Updated dependency to the latest version of the workflow core package.	+2/-2
package.json `Update Package Version and Dependencies for Workflows Service` services/workflows-service/package.json Updated version to 0.7.6. Updated dependencies to include new versions of common, core, and node SDK packages.	+4/-4
package.json `Update Dependency for Documentation Website` websites/docs/package.json Updated dependency to the latest version of the common package.	+1/-1

✨ PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

changeset-bot · 2024-04-22T07:50:32Z

⚠️ No Changeset found

Latest commit: ca4d26e

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

coderabbitai · 2024-04-22T07:50:43Z

Walkthrough

This update primarily focuses on bumping versions and updating dependencies across several packages including @ballerine/common, @ballerine/workflow-core, and various SDKs and apps. Additionally, new document schemas for financial and registration information have been introduced in the @ballerine/common package for Ghana and Uganda.

Changes

File Path	Change Summary
`.../backoffice-v2/package.json`, `.../kyb-app/package.json`, `.../headless-example/package.json`, `.../common/package.json`, `.../workflow-core/package.json`, `.../web-ui-sdk/package.json`, `.../workflow-browser-sdk/package.json`, `.../workflow-node-sdk/package.json`, `.../workflows-service/package.json`, `.../docs/package.json`	Updated package versions and dependencies.
`.../common/src/schemas/.../GH.ts`, `.../common/src/schemas/.../UG.ts`	Added new document schemas for Ghana and Uganda.
`.../workflows-service/prisma/data-migrations`	Updated subproject commit hash.

🐰✨📜
In the land of code and schema,
A rabbit hopped with a gleeful dilemma.
Bumping versions, weaving dependencies tight,
New schemas bloom, a developer's delight!
Hop, skip, a commit, push with glee,
Oh, what a versioned world this will be! 🌟🐾

Recent Review Details

Configuration used: CodeRabbit UI
Review profile: CHILL

Commits

Files that changed from the base of the PR and between 89e49fd and ca4d26e.

Files ignored due to path filters (1)

pnpm-lock.yaml is excluded by !pnpm-lock.yaml

Files selected for processing (22)

apps/backoffice-v2/CHANGELOG.md (1 hunks)
apps/backoffice-v2/package.json (2 hunks)
apps/kyb-app/CHANGELOG.md (1 hunks)
apps/kyb-app/package.json (2 hunks)
examples/headless-example/CHANGELOG.md (1 hunks)
examples/headless-example/package.json (2 hunks)
packages/common/CHANGELOG.md (1 hunks)
packages/common/package.json (1 hunks)
packages/common/src/schemas/documents/workflow/documents/schemas/GH.ts (8 hunks)
packages/common/src/schemas/documents/workflow/documents/schemas/UG.ts (8 hunks)
packages/workflow-core/CHANGELOG.md (1 hunks)
packages/workflow-core/package.json (2 hunks)
sdks/web-ui-sdk/CHANGELOG.md (1 hunks)
sdks/web-ui-sdk/package.json (2 hunks)
sdks/workflow-browser-sdk/CHANGELOG.md (1 hunks)
sdks/workflow-browser-sdk/package.json (2 hunks)
sdks/workflow-node-sdk/CHANGELOG.md (1 hunks)
sdks/workflow-node-sdk/package.json (2 hunks)
services/workflows-service/CHANGELOG.md (1 hunks)
services/workflows-service/package.json (2 hunks)
services/workflows-service/prisma/data-migrations (1 hunks)
websites/docs/package.json (1 hunks)

Files skipped from review due to trivial changes (2)

packages/common/package.json
services/workflows-service/prisma/data-migrations

Additional Context Used

LanguageTool (286)

apps/backoffice-v2/CHANGELOG.md (25)

Near line 49: Possible spelling mistake found.
Context: ...k@0.6.3 ## 0.7.1 ### Patch Changes - bumo - Updated dependencies - @ballerine/w...

Near line 65: Possible spelling mistake found.
Context: ...- 9fe7a5c: Bump ### Patch Changes - ver bump - 9fe7a5c: bump alpha - 9fe7a5c: bu...

Near line 69: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...a5c10: bump - 9fe7a5c: Version bump - Updated dependencies - Updated dependencies [9fe7a5c] - Updated dependencies [9fe...

Near line 196: Possible spelling mistake found.
Context: ...0.5.44 ## 0.5.49 ### Patch Changes - Ver bump - Updated dependencies - @ballerine/b...

Near line 384: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...0.5.27 ## 0.5.29 ### Patch Changes - Updated dependencies - Updated dependencies [59afd0b] - @ballerine/common@0.7.27...

Near line 490: Possible spelling mistake found.
Context: ...0 ## 0.5.18 ### Patch Changes - Bump ver - Updated dependencies - @ballerine/c...

Near line 520: Possible spelling mistake found.
Context: ...0.5.16 ## 0.5.15 ### Patch Changes - AsiaVerify integrations - Updated dependencies -...

Near line 531: Possible spelling mistake found.
Context: ...ch Changes - 801fc63: pre - 801fc63: Pre ver - 801fc63: Prerelease - 801fc63: bump...

Near line 538: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...: Pre bump - 801fc63: Bump - 801fc63: pre - Version bump - 801fc63: Pre release - 801fc63...

Near line 541: Possible spelling mistake found.
Context: ...ump - 801fc63: Pre release - 801fc63: Bumo - Updated dependencies [801fc63] - Upd...

Near line 557: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...39] - Updated dependencies [801fc63] - Updated dependencies - Updated dependencies [801fc63] - Updated dependencies [801f...

Near line 605: Possible spelling mistake found.
Context: ...ch Changes - b1cebf5: pre - b1cebf5: Pre ver - b1cebf5: Prerelease - b1cebf5: bump...

Near line 608: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...bf50: bump pre - b1cebf5: Version bump pre - Bump - b1cebf5: Pre bump - b1cebf5: Pre bu...

Near line 615: Possible spelling mistake found.
Context: ...pre - b1cebf5: Pre release - b1cebf5: Bumo - Updated dependencies [b1cebf5] - Upd...

Near line 622: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...50] - Updated dependencies [b1cebf5] - Updated dependencies - Updated dependencies [b1cebf5] - Updated dependencies [b1ce...

Near line 642: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...8.15 ### Patch Changes - Version bump pre - Updated dependencies - @ballerine/workflow-br...

Near line 664: Possible spelling mistake found.
Context: ....5.11-3e08f108.13 ### Patch Changes - Bumo - Updated dependencies - @ballerine/w...

Near line 682: Possible spelling mistake found.
Context: ....5.11-3e08f108.11 ### Patch Changes - Pre ver - Updated dependencies - @ballerine/w...

Near line 704: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...1-3e08f108.9 ### Patch Changes - bump pre - Updated dependencies - @ballerine/workflow-br...

Near line 740: This expression is normally spelled as one or with a hyphen.
Context: ...0.5.11-3e08f108.5 ### Patch Changes - Pre release - Updated dependencies - @ballerine/w...

Near line 765: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...0.5.11-3e08f108.2 ### Patch Changes - pre - Updated dependencies - @ballerine/workflow-br...

Near line 776: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...0.5.11-3e08f108.1 ### Patch Changes - pre - Updated dependencies - @ballerine/workflow-br...

Near line 835: Possible typo detected.
Context: ... - Version bump - Updated dependencies [efedc9c] - Updated dependencies - @ballerine/...

Near line 942: Possible typo detected.
Context: ... Patch Changes - Updated dependencies [b8dedaa] - @ballerine/ui@0.2.13 ## 0.4.13 #...

Near line 969: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...> demo ## 0.4.11 ### Patch Changes - Updated dependencies - Updated dependencies [cce4a66] - @ballerine/ui@0.2.10 -...

apps/kyb-app/CHANGELOG.md (33)

Near line 1: Possible spelling mistake found.
Context: # kyb-app ## 0.3.7 ### Patch Changes - Updated...

Near line 50: Possible spelling mistake found.
Context: ...k@0.6.3 ## 0.3.1 ### Patch Changes - bumo - Updated dependencies - @ballerine/w...

Near line 65: Possible spelling mistake found.
Context: ...- 9fe7a5c: Bump ### Patch Changes - ver bump - 9fe7a5c: bump - 9fe7a5c: Version ...

Near line 68: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...a5c10: bump - 9fe7a5c: Version bump - Updated dependencies - Updated dependencies [9fe7a5c] - Updated dependencies [9fe...

Near line 183: Possible spelling mistake found.
Context: ...0.5.44 ## 0.1.44 ### Patch Changes - Ver bump - Updated dependencies - @ballerine/b...

Near line 364: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...0.5.27 ## 0.1.23 ### Patch Changes - Updated dependencies - Updated dependencies [59afd0b] - @ballerine/common@0.7.27...

Near line 463: Possible spelling mistake found.
Context: ...0 ## 0.1.12 ### Patch Changes - Bump ver - Updated dependencies - @ballerine/b...

Near line 489: Possible spelling mistake found.
Context: ... dependencies - @ballerine/ui@0.3.8 - AsiaVerify integrations - @ballerine/blocks@0.1....

Near line 498: Possible spelling mistake found.
Context: ...ch Changes - 801fc63: pre - 801fc63: Pre ver - 801fc63: Prerelease - 801fc63: bump...

Near line 505: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...: Pre bump - 801fc63: Bump - 801fc63: pre - Version bump - 801fc63: Pre release - 801fc63...

Near line 508: Possible spelling mistake found.
Context: ...ump - 801fc63: Pre release - 801fc63: Bumo - Updated dependencies [801fc63] - Upd...

Near line 524: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...39] - Updated dependencies [801fc63] - Updated dependencies - Updated dependencies [801fc63] - Updated dependencies [801f...

Near line 568: Possible spelling mistake found.
Context: ...ch Changes - b1cebf5: pre - b1cebf5: Pre ver - b1cebf5: Prerelease - b1cebf5: bump...

Near line 571: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...bf50: bump pre - b1cebf5: Version bump pre - Bump - b1cebf5: Pre bump - b1cebf5: Pre bu...

Near line 578: Possible spelling mistake found.
Context: ...pre - b1cebf5: Pre release - b1cebf5: Bumo - Updated dependencies [b1cebf5] - Upd...

Near line 585: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...50] - Updated dependencies [b1cebf5] - Updated dependencies - Updated dependencies [b1cebf5] - Updated dependencies [b1ce...

Near line 604: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...8.15 ### Patch Changes - Version bump pre - Updated dependencies - @ballerine/workflow-br...

Near line 624: Possible spelling mistake found.
Context: ...0.1.5-3e08f108.13 ### Patch Changes - Bumo - Updated dependencies - @ballerine/w...

Near line 641: Possible spelling mistake found.
Context: ...0.1.5-3e08f108.11 ### Patch Changes - Pre ver - Updated dependencies - @ballerine/w...

Near line 661: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...5-3e08f108.9 ### Patch Changes - bump pre - Updated dependencies - @ballerine/workflow-br...

Near line 695: This expression is normally spelled as one or with a hyphen.
Context: ... 0.1.5-3e08f108.5 ### Patch Changes - Pre release - Updated dependencies - @ballerine/w...

Near line 719: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ... 0.1.5-3e08f108.2 ### Patch Changes - pre - Updated dependencies - @ballerine/workflow-br...

Near line 729: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ... 0.1.5-3e08f108.1 ### Patch Changes - pre - Updated dependencies - @ballerine/workflow-br...

Near line 818: Possible typo detected.
Context: ... Patch Changes - Updated dependencies [b8dedaa] - @ballerine/ui@0.2.13 ## 0.0.19 #...

Near line 869: Possible spelling mistake found.
Context: ...3 ### Patch Changes - c06f234: added DynamicForm & TextArea & css fixes - Updated depend...

Near line 869: Possible spelling mistake found.
Context: ...hanges - c06f234: added DynamicForm & TextArea & css fixes - Updated dependencies - Up...

Near line 869: File types are normally capitalized.
Context: ...06f2341: added DynamicForm & TextArea & css fixes - Updated dependencies - Updated ...

Near line 870: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...ed DynamicForm & TextArea & css fixes - Updated dependencies - Updated dependencies [c06f234] - @ballerine/ui@0.2.3 ## ...

Near line 885: Put a space after the comma.
Context: ... ### Patch Changes - added Autocomplete,DropdownInput,Multiselect,DatePicker,Checkbox,PhoneNu...

Near line 885: Put a space after the comma.
Context: ...nges - added Autocomplete,DropdownInput,Multiselect,DatePicker,Checkbox,PhoneNumber compone...

Near line 885: Put a space after the comma.
Context: ...d Autocomplete,DropdownInput,Multiselect,DatePicker,Checkbox,PhoneNumber components - Updat...

Near line 885: Put a space after the comma.
Context: ...ete,DropdownInput,Multiselect,DatePicker,Checkbox,PhoneNumber components - Updated depend...

Near line 885: Possible spelling mistake found.
Context: ...wnInput,Multiselect,DatePicker,Checkbox,PhoneNumber components - Updated dependencies - @...

examples/headless-example/CHANGELOG.md (27)

Near line 43: Possible spelling mistake found.
Context: ...k@0.6.3 ## 0.3.1 ### Patch Changes - bumo - Updated dependencies - @ballerine/w...

Near line 56: Possible spelling mistake found.
Context: ...- 9fe7a5c: Bump ### Patch Changes - ver bump - 9fe7a5c: bump - 9fe7a5c: Version ...

Near line 59: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...a5c10: bump - 9fe7a5c: Version bump - Updated dependencies - Updated dependencies [9fe7a5c] - Updated dependencies [9fe...

Near line 164: Possible spelling mistake found.
Context: ...0.5.44 ## 0.1.43 ### Patch Changes - Ver bump - Updated dependencies - @ballerine/c...

Near line 303: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...0.5.27 ## 0.1.26 ### Patch Changes - Updated dependencies - Updated dependencies [59afd0b] - @ballerine/common@0.7.27...

Near line 364: Possible spelling mistake found.
Context: ...9 ## 0.1.18 ### Patch Changes - Bump ver - Updated dependencies - @ballerine/c...

Near line 398: Possible spelling mistake found.
Context: ...ch Changes - 801fc63: pre - 801fc63: Pre ver - 801fc63: Prerelease - 801fc63: bump...

Near line 405: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...: Pre bump - 801fc63: Bump - 801fc63: pre - Version bump - 801fc63: Pre release - 801fc63...

Near line 408: Possible spelling mistake found.
Context: ...ump - 801fc63: Pre release - 801fc63: Bumo - Updated dependencies [801fc63] - Upd...

Near line 418: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...39] - Updated dependencies [801fc63] - Updated dependencies - Updated dependencies [801fc63] - Updated dependencies [801f...

Near line 446: Possible spelling mistake found.
Context: ...ch Changes - b1cebf5: pre - b1cebf5: Pre ver - b1cebf5: Prerelease - b1cebf5: bump...

Near line 449: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...bf50: bump pre - b1cebf5: Version bump pre - Bump - b1cebf5: Pre bump - b1cebf5: Pre bu...

Near line 456: Possible spelling mistake found.
Context: ...pre - b1cebf5: Pre release - b1cebf5: Bumo - Updated dependencies [b1cebf5] - Upd...

Near line 462: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...50] - Updated dependencies [b1cebf5] - Updated dependencies - Updated dependencies [b1cebf5] - Updated dependencies [b1ce...

Near line 476: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...8.10 ### Patch Changes - Version bump pre - Updated dependencies - @ballerine/workflow-br...

Near line 494: Possible spelling mistake found.
Context: ...0.1.11-3e08f108.8 ### Patch Changes - Bumo - Updated dependencies - @ballerine/w...

Near line 503: Possible spelling mistake found.
Context: ...0.1.11-3e08f108.7 ### Patch Changes - Pre ver - Updated dependencies - @ballerine/w...

Near line 521: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...1-3e08f108.5 ### Patch Changes - bump pre - Updated dependencies - @ballerine/workflow-br...

Near line 539: This expression is normally spelled as one or with a hyphen.
Context: ...0.1.11-3e08f108.3 ### Patch Changes - Pre release - Updated dependencies - @ballerine/w...

Near line 548: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...0.1.11-3e08f108.2 ### Patch Changes - pre - Updated dependencies - @ballerine/workflow-br...

Near line 557: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...0.1.11-3e08f108.1 ### Patch Changes - pre - Updated dependencies - @ballerine/workflow-br...

Near line 601: Possible typo detected.
Context: ... - Version bump - Updated dependencies [efedc9c] - Updated dependencies - @ballerine/...

Near line 728: Put a space after the comma.
Context: ... ### Patch Changes - added Autocomplete,DropdownInput,Multiselect,DatePicker,Checkbox,PhoneNu...

Near line 728: Put a space after the comma.
Context: ...nges - added Autocomplete,DropdownInput,Multiselect,DatePicker,Checkbox,PhoneNumber compone...

Near line 728: Put a space after the comma.
Context: ...d Autocomplete,DropdownInput,Multiselect,DatePicker,Checkbox,PhoneNumber components - Updat...

Near line 728: Put a space after the comma.
Context: ...ete,DropdownInput,Multiselect,DatePicker,Checkbox,PhoneNumber components - Updated depend...

Near line 728: Possible spelling mistake found.
Context: ...wnInput,Multiselect,DatePicker,Checkbox,PhoneNumber components - Updated dependencies - @...

packages/common/CHANGELOG.md (39)

Near line 25: Possible spelling mistake found.
Context: ...schemas ## 0.9.1 ### Patch Changes - bumo ## 0.9.0 ### Minor Changes - 9fe7a5c...

Near line 35: Possible spelling mistake found.
Context: ...- 9fe7a5c: Bump ### Patch Changes - ver bump - 9fe7a5c: bump - 9fe7a5c: Added ba...

Near line 81: Possible spelling mistake found.
Context: ...45 ### Patch Changes - Updated common enums ## 0.7.44 ### Patch Changes - Fixes ...

Near line 93: Possible spelling mistake found.
Context: ... fixes ## 0.7.43 ### Patch Changes - Ver bump ## 0.7.42 ### Patch Changes - update...

Near line 147: Possible spelling mistake found.
Context: ...## 0.7.34 ### Patch Changes - updated DefaultContextSchema type to use Typebox over deprecated gen...

Near line 147: Possible spelling mistake found.
Context: ...pdated DefaultContextSchema type to use Typebox over deprecated generation script ## 0...

Near line 153: Possible spelling mistake found.
Context: ...ch Changes - added default context for defintion testing ## 0.7.32 ### Patch Changes ...

Near line 159: Possible spelling mistake found.
Context: ...atch Changes - add example context for defintion validation ## 0.7.31 ### Patch Change...

Near line 177: File types are normally capitalized.
Context: ...9 ### Patch Changes - 340d9d4: added xlsx and xls to file type schema - ea423a2:...

Near line 177: File types are normally capitalized.
Context: ...tch Changes - 340d9d4: added xlsx and xls to file type schema - ea423a2: updated...

Near line 198: Possible spelling mistake found.
Context: ...ull with undefined - updated version of defintion ## 0.7.25 ### Patch Changes - update...

Near line 210: Possible spelling mistake found.
Context: ...## 0.7.24 ### Patch Changes - Removed imageUrl property from ZZ document schemas - d5c...

Near line 210: Possible spelling mistake found.
Context: ...anges - Removed imageUrl property from ZZ document schemas - d5cca1e: added defi...

Near line 211: Possible spelling mistake found.
Context: ...m ZZ document schemas - d5cca1e: added defintion validation - f4b5c0d: added defintion ...

Near line 212: Possible spelling mistake found.
Context: ... defintion validation - f4b5c0d: added defintion validation ## 0.7.23 ### Patch Change...

Near line 218: Possible spelling mistake found.
Context: ... ### Patch Changes - 8c888ce: Updated tsconfig and types ## 0.7.22 ### Patch Changes...

Near line 224: Possible spelling mistake found.
Context: ... ## 0.7.22 ### Patch Changes - Added ZZ schemas - Change document schemas ## 0...

Near line 237: Possible spelling mistake found.
Context: ...p ## 0.7.20 ### Patch Changes - Bump ver ## 0.7.19 ### Patch Changes - Fixed ...

Near line 255: Possible spelling mistake found.
Context: ...schema ## 0.7.17 ### Patch Changes - AsiaVerify integrations ## 0.7.16 ### Patch Chan...

Near line 262: Possible spelling mistake found.
Context: ...ch Changes - 801fc63: pre - 801fc63: Pre ver - 801fc63: bump pre - 801fc63: Versio...

Near line 268: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...: Pre bump - 801fc63: Bump - 801fc63: pre - Version bump - 801fc63: Pre release - 801fc63...

Near line 271: Possible spelling mistake found.
Context: ...ump - 801fc63: Pre release - 801fc63: Bumo ## 0.7.15 ### Patch Changes - 306ca4...

Near line 277: Possible spelling mistake found.
Context: ... 0.7.15 ### Patch Changes - 306ca46: businessName now may accept spaces ## 0.7.14 ### P...

Near line 290: Possible spelling mistake found.
Context: ...ch Changes - b1cebf5: pre - b1cebf5: Pre ver - b1cebf5: bump pre - b1cebf5: Versio...

Near line 292: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...bf50: bump pre - b1cebf5: Version bump pre - Bump - b1cebf5: Pre bump - b1cebf5: Pre bu...

Near line 299: Possible spelling mistake found.
Context: ...pre - b1cebf5: Pre release - b1cebf5: Bumo ## 0.7.13-3e08f108.9 ### Patch Change...

Near line 317: Possible spelling mistake found.
Context: ...0.7.13-3e08f108.7 ### Patch Changes - Bumo ## 0.7.13-3e08f108.6 ### Patch Change...

Near line 323: Possible spelling mistake found.
Context: ...0.7.13-3e08f108.6 ### Patch Changes - Pre ver ## 0.7.13-3e08f108.5 ### Patch Change...

Near line 347: This expression is normally spelled as one or with a hyphen.
Context: ...0.7.13-3e08f108.2 ### Patch Changes - Pre release ## 0.7.13-3e08f108.1 ### Patch Change...

Near line 377: Possible typo detected.
Context: ...r bump ## 0.7.10 ### Patch Changes - efedc9c: Update document schemas - Version bump...

Near line 426: Possible spelling mistake found.
Context: ...5 ### Patch Changes - c2a1380: Added StateTags ## 0.5.24 ### Patch Changes - 204b54...

Near line 438: Possible spelling mistake found.
Context: ...## 0.5.23 ### Patch Changes - Removed verifiySignature util ## 0.5.22 ### Patch Changes - A...

Near line 444: Possible spelling mistake found.
Context: ... ## 0.5.22 ### Patch Changes - Added verifySignature function ## 0.5.20 ### Patch Changes...

Near line 493: Put a space after the comma.
Context: ... ### Patch Changes - added Autocomplete,DropdownInput,Multiselect,DatePicker,Checkbox,PhoneNu...

Near line 493: Put a space after the comma.
Context: ...nges - added Autocomplete,DropdownInput,Multiselect,DatePicker,Checkbox,PhoneNumber compone...

Near line 493: Put a space after the comma.
Context: ...d Autocomplete,DropdownInput,Multiselect,DatePicker,Checkbox,PhoneNumber components - Upgra...

Near line 493: Put a space after the comma.
Context: ...ete,DropdownInput,Multiselect,DatePicker,Checkbox,PhoneNumber components - Upgrade packag...

Near line 493: Possible spelling mistake found.
Context: ...wnInput,Multiselect,DatePicker,Checkbox,PhoneNumber components - Upgrade packages ## 0.5.1...

Near line 576: Possible spelling mistake found.
Context: ...validation ### Patch Changes - Adding scheams ## 0.4.3 ### Patch Changes - a2bed5e...

packages/workflow-core/CHANGELOG.md (35)

Near line 28: Possible spelling mistake found.
Context: ...2 ## 0.6.4 ### Patch Changes - Fixed invokedAt to be UTC epoch ## 0.6.3 ### Patch Ch...

Near line 40: Possible spelling mistake found.
Context: ... logger ## 0.6.2 ### Patch Changes - bumo - Updated dependencies - @ballerine/c...

Near line 48: Possible spelling mistake found.
Context: ...n@0.9.1 ## 0.6.1 ### Patch Changes - ver bump - 9fe7a5c: bump - Updated dependencie...

Near line 50: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...Changes - ver bump - 9fe7a5c: bump - Updated dependencies - Updated dependencies [9fe7a5c] - Updated dependencies [9fe...

Near line 92: Possible spelling mistake found.
Context: ... ## 0.5.50 ### Patch Changes - fixed invokedAt ## 0.5.49 ### Patch Changes - versio...

Near line 144: Possible spelling mistake found.
Context: ...0.7.44 ## 0.5.43 ### Patch Changes - Ver bump - Updated dependencies - @ballerine/c...

Near line 189: Possible spelling mistake found.
Context: ...### Patch Changes - Better logging for api plugin ## 0.5.36 ### Patch Changes -...

Near line 269: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...0.7.28 ## 0.5.26 ### Patch Changes - Updated dependencies - Updated dependencies [59afd0b] - @ballerine/common@0.7.27...

Near line 292: Possible spelling mistake found.
Context: ... ## 0.5.23 ### Patch Changes - Added TransformerPlugin - d5cca1e: added defintion validation ...

Near line 293: Possible spelling mistake found.
Context: ...ded TransformerPlugin - d5cca1e: added defintion validation - f4b5c0d: added defintion ...

Near line 294: Possible spelling mistake found.
Context: ... defintion validation - f4b5c0d: added defintion validation - Updated dependencies [d5cc...

Near line 303: Possible spelling mistake found.
Context: ... ### Patch Changes - 8c888ce: Updated tsconfig and types - Updated dependencies [8c888...

Near line 333: Possible spelling mistake found.
Context: ...1 ## 0.5.18 ### Patch Changes - Bump ver - Updated dependencies - @ballerine/c...

Near line 355: Possible spelling mistake found.
Context: ...0.7.18 ## 0.5.15 ### Patch Changes - AsiaVerify integrations - Updated dependencies -...

Near line 364: Possible spelling mistake found.
Context: ...ch Changes - 801fc63: pre - 801fc63: Pre ver - 801fc63: Prerelease - 801fc63: bump...

Near line 371: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...: Pre bump - 801fc63: Bump - 801fc63: pre - Version bump - 801fc63: Pre release - 801fc63...

Near line 374: Possible spelling mistake found.
Context: ...ump - 801fc63: Pre release - 801fc63: Bumo - Updated dependencies [801fc63] - Upd...

Near line 383: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...39] - Updated dependencies [801fc63] - Updated dependencies - Updated dependencies [801fc63] - Updated dependencies [801f...

Near line 408: Possible spelling mistake found.
Context: ...ch Changes - b1cebf5: pre - b1cebf5: Pre ver - b1cebf5: Prerelease - b1cebf5: bump...

Near line 411: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...bf50: bump pre - b1cebf5: Version bump pre - Bump - b1cebf5: Pre bump - b1cebf5: Pre bu...

Near line 418: Possible spelling mistake found.
Context: ...pre - b1cebf5: Pre release - b1cebf5: Bumo - Updated dependencies [b1cebf5] - Upd...

Near line 423: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...50] - Updated dependencies [b1cebf5] - Updated dependencies - Updated dependencies [b1cebf5] - Updated dependencies [b1ce...

Near line 436: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...8.10 ### Patch Changes - Version bump pre - Updated dependencies - @ballerine/common@0.7....

Near line 452: Possible spelling mistake found.
Context: ...0.5.11-3e08f108.8 ### Patch Changes - Bumo - Updated dependencies - @ballerine/c...

Near line 460: Possible spelling mistake found.
Context: ...0.5.11-3e08f108.7 ### Patch Changes - Pre ver - Updated dependencies - @ballerine/c...

Near line 476: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...1-3e08f108.5 ### Patch Changes - bump pre - Updated dependencies - @ballerine/common@0.7....

Near line 492: This expression is normally spelled as one or with a hyphen.
Context: ...0.5.11-3e08f108.3 ### Patch Changes - Pre release - Updated dependencies - @ballerine/c...

Near line 500: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...0.5.11-3e08f108.2 ### Patch Changes - pre - Updated dependencies - @ballerine/common@0.7....

Near line 508: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...0.5.11-3e08f108.1 ### Patch Changes - pre - Updated dependencies - @ballerine/common@0.7....

Near line 546: Possible typo detected.
Context: ... - Version bump - Updated dependencies [efedc9c] - Updated dependencies - @ballerine/...

Near line 777: Possible spelling mistake found.
Context: ...Patch Changes - 700d492: added missing changeset - fixed monorepo infra ## 0.4.5 ### P...

Near line 777: To make your text as clear as possible to all readers, do not use this foreign term. Possible alternatives are “below” or “further on” (in a document).
Context: ...dded missing changeset - fixed monorepo infra ## 0.4.5 ### Patch Changes - build -...

Near line 783: Do you wish to insert an arrow?
Context: ...a ## 0.4.5 ### Patch Changes - build -> dist ## 0.4.4 ### Patch Changes - c0...

Near line 790: Possible spelling mistake found.
Context: ...f88: removed previous implementation of statePlugins - c018f88: workflow-core consumers may ...

Near line 799: Possible spelling mistake found.
Context: ...b10: removed previous implementation of statePlugins - 3bb5b10: added state plugins, actions...

sdks/web-ui-sdk/CHANGELOG.md (36)

Near line 1: Possible spelling mistake found.
Context: # web-ui-sdk ## 1.5.5 ### Patch Changes - Updated...

Near line 28: Possible spelling mistake found.
Context: ...n@0.9.2 ## 1.5.2 ### Patch Changes - bumo - Updated dependencies - @ballerine/c...

Near line 36: Possible spelling mistake found.
Context: ...n@0.9.1 ## 1.5.1 ### Patch Changes - ver bump - 9fe7a5c: bump - Updated dependencie...

Near line 38: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...Changes - ver bump - 9fe7a5c: bump - Updated dependencies - Updated dependencies [9fe7a5c] - Updated dependencies [9fe...

Near line 112: Possible spelling mistake found.
Context: ...0.7.44 ## 1.4.41 ### Patch Changes - Ver bump - Updated dependencies - @ballerine/c...

Near line 229: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...0.7.28 ## 1.4.24 ### Patch Changes - Updated dependencies - Updated dependencies [59afd0b] - @ballerine/common@0.7.27...

Near line 283: Possible spelling mistake found.
Context: ...1 ## 1.4.17 ### Patch Changes - Bump ver - Updated dependencies - @ballerine/c...

Near line 313: Possible spelling mistake found.
Context: ...ch Changes - 801fc63: pre - 801fc63: Pre ver - 801fc63: bump pre - 801fc63: Versio...

Near line 319: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...: Pre bump - 801fc63: Bump - 801fc63: pre - Version bump - 801fc63: Pre release - 801fc63...

Near line 322: Possible spelling mistake found.
Context: ...ump - 801fc63: Pre release - 801fc63: Bumo - Updated dependencies [801fc63] - Upd...

Near line 331: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...39] - Updated dependencies [801fc63] - Updated dependencies - Updated dependencies [801fc63] - Updated dependencies [801f...

Near line 356: Possible spelling mistake found.
Context: ...ch Changes - b1cebf5: pre - b1cebf5: Pre ver - b1cebf5: bump pre - b1cebf5: Versio...

Near line 358: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...bf50: bump pre - b1cebf5: Version bump pre - Bump - b1cebf5: Pre bump - b1cebf5: Pre bu...

Near line 365: Possible spelling mistake found.
Context: ...pre - b1cebf5: Pre release - b1cebf5: Bumo - Updated dependencies [b1cebf5] - Upd...

Near line 370: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...50] - Updated dependencies [b1cebf5] - Updated dependencies - Updated dependencies [b1cebf5] - Updated dependencies [b1ce...

Near line 383: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...08.9 ### Patch Changes - Version bump pre - Updated dependencies - @ballerine/common@0.7....

Near line 399: Possible spelling mistake found.
Context: ...1.4.10-3e08f108.7 ### Patch Changes - Bumo - Updated dependencies - @ballerine/c...

Near line 407: Possible spelling mistake found.
Context: ...1.4.10-3e08f108.6 ### Patch Changes - Pre ver - Updated dependencies - @ballerine/c...

Near line 423: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...0-3e08f108.4 ### Patch Changes - bump pre - Updated dependencies - @ballerine/common@0.7....

Near line 439: This expression is normally spelled as one or with a hyphen.
Context: ...1.4.10-3e08f108.2 ### Patch Changes - Pre release - Updated dependencies - @ballerine/c...

Near line 447: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...1.4.10-3e08f108.1 ### Patch Changes - pre - Updated dependencies - @ballerine/common@0.7....

Near line 455: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...1.4.10-3e08f108.0 ### Patch Changes - pre - Updated dependencies - @ballerine/common@0.7....

Near line 480: Possible typo detected.
Context: ... - Version bump - Updated dependencies [efedc9c] - Updated dependencies - @ballerine/...

Near line 689: Possible spelling mistake found.
Context: ... Svelte stores as the first step of the init method ## 1.2.3 ### Patch Changes - ...

Near line 695: Possible spelling mistake found.
Context: ...## 1.2.3 ### Patch Changes - a24a4cc: Init now handles multiple calls by resetting...

Near line 707: Possible spelling mistake found.
Context: ...1 ### Patch Changes - Adding advanced ui configuration capabilities ## 1.2.0 #...

Near line 713: Possible spelling mistake found.
Context: ...r Changes - UI packs capabilities, new ui packs (future) ## 1.1.38 ### Patch Ch...

Near line 719: Possible spelling mistake found.
Context: ...# 1.1.38 ### Patch Changes - UI-Packs improvments ## 1.1.37 ### Patch Changes - Fixed ...

Near line 725: File types are normally capitalized.
Context: ... ## 1.1.37 ### Patch Changes - Fixed svg issue oin iOS ## 1.1.36 ### Patch Cha...

Near line 725: Possible spelling mistake found.
Context: ...7 ### Patch Changes - Fixed svg issue oin iOS ## 1.1.36 ### Patch Changes - En...

Near line 775: Possible spelling mistake found.
Context: ... 1.1.29 ### Patch Changes - UI Fixes, kyb flow fix ## 1.1.28 ### Patch Changes ...

Near line 787: Possible spelling mistake found.
Context: ...hanges ## 1.1.27 ### Patch Changes - Refactoing ## 1.1.26 ### Patch Changes - Bug fi...

Near line 799: Possible spelling mistake found.
Context: ... ### Patch Changes - 615f3d0: renamed web-ui-sdk to @ballerine/web-ui-sdk ## 1.1.24 ##...

Near line 799: Possible spelling mistake found.
Context: ...5f3d0: renamed web-ui-sdk to @ballerine/web-ui-sdk ## 1.1.24 ### Patch Changes - 750687...

Near line 805: Possible spelling mistake found.
Context: ...24 ### Patch Changes - 7506870: moved changesets to root of monorepo ## 1.1.23 ### Pat...

Near line 811: Possible spelling mistake found.
Context: ...23 ### Patch Changes - a1d5718: moved changesets to root of monorepo

sdks/workflow-browser-sdk/CHANGELOG.md (34)

Near line 1: Possible spelling mistake found.
Context: # @ballerine/workflow-browser-sdk ## 0.6.7 ### Patch Changes - Updated...

Near line 38: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...e@0.6.4 ## 0.6.3 ### Patch Changes - Updated dependencies - Updated dependencies [0244d3e] - @ballerine/workflow-cor...

Near line 46: Possible spelling mistake found.
Context: ...e@0.6.3 ## 0.6.2 ### Patch Changes - bumo - Updated dependencies - @ballerine/w...

Near line 55: Possible spelling mistake found.
Context: ...n@0.9.1 ## 0.6.1 ### Patch Changes - ver bump - 9fe7a5c: bump - Updated dependencie...

Near line 57: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...Changes - ver bump - 9fe7a5c: bump - Updated dependencies - Updated dependencies [9fe7a5c] - Updated dependencies [9fe...

Near line 161: Possible spelling mistake found.
Context: ...0.7.44 ## 0.5.43 ### Patch Changes - Ver bump - Updated dependencies - @ballerine/c...

Near line 329: Possible spelling mistake found.
Context: ...3 ### Patch Changes - d5cca1e: added defintion validation - Updated dependencies [d5cc...

Near line 339: Possible spelling mistake found.
Context: ... ### Patch Changes - 8c888ce: Updated tsconfig and types - Updated dependencies [8c888...

Near line 374: Possible spelling mistake found.
Context: ...9 ## 0.5.18 ### Patch Changes - Bump ver - Updated dependencies - @ballerine/c...

Near line 408: Possible spelling mistake found.
Context: ...ch Changes - 801fc63: pre - 801fc63: Pre ver - 801fc63: Prerelease - 801fc63: bump...

Near line 415: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...: Pre bump - 801fc63: Bump - 801fc63: pre - Version bump - 801fc63: Pre release - 801fc63...

Near line 418: Possible spelling mistake found.
Context: ...ump - 801fc63: Pre release - 801fc63: Bumo - Updated dependencies [801fc63] - Upd...

Near line 428: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...39] - Updated dependencies [801fc63] - Updated dependencies - Updated dependencies [801fc63] - Updated dependencies [801f...

Near line 456: Possible spelling mistake found.
Context: ...ch Changes - b1cebf5: pre - b1cebf5: Pre ver - b1cebf5: Prerelease - b1cebf5: bump...

Near line 459: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...bf50: bump pre - b1cebf5: Version bump pre - Bump - b1cebf5: Pre bump - b1cebf5: Pre bu...

Near line 466: Possible spelling mistake found.
Context: ...pre - b1cebf5: Pre release - b1cebf5: Bumo - Updated dependencies [b1cebf5] - Upd...

Near line 472: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...50] - Updated dependencies [b1cebf5] - Updated dependencies - Updated dependencies [b1cebf5] - Updated dependencies [b1ce...

Near line 486: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...8.10 ### Patch Changes - Version bump pre - Updated dependencies - @ballerine/workflow-co...

Near line 504: Possible spelling mistake found.
Context: ...0.5.11-3e08f108.8 ### Patch Changes - Bumo - Updated dependencies - @ballerine/w...

Near line 513: Possible spelling mistake found.
Context: ...0.5.11-3e08f108.7 ### Patch Changes - Pre ver - Updated dependencies - @ballerine/w...

Near line 531: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...1-3e08f108.5 ### Patch Changes - bump pre - Updated dependencies - @ballerine/workflow-co...

Near line 549: This expression is normally spelled as one or with a hyphen.
Context: ...0.5.11-3e08f108.3 ### Patch Changes - Pre release - Updated dependencies - @ballerine/w...

Near line 558: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...0.5.11-3e08f108.2 ### Patch Changes - pre - Updated dependencies - @ballerine/workflow-co...

Near line 567: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...0.5.11-3e08f108.1 ### Patch Changes - pre - Updated dependencies - @ballerine/workflow-co...

Near line 613: Possible typo detected.
Context: ... - Version bump - Updated dependencies [efedc9c] - Updated dependencies - @ballerine/...

Near line 741: Put a space after the comma.
Context: ... ### Patch Changes - added Autocomplete,DropdownInput,Multiselect,DatePicker,Checkbox,PhoneNu...

Near line 741: Put a space after the comma.
Context: ...nges - added Autocomplete,DropdownInput,Multiselect,DatePicker,Checkbox,PhoneNumber compone...

Near line 741: Put a space after the comma.
Context: ...d Autocomplete,DropdownInput,Multiselect,DatePicker,Checkbox,PhoneNumber components - Updat...

Near line 741: Put a space after the comma.
Context: ...ete,DropdownInput,Multiselect,DatePicker,Checkbox,PhoneNumber components - Updated depend...

Near line 741: Possible spelling mistake found.
Context: ...wnInput,Multiselect,DatePicker,Checkbox,PhoneNumber components - Updated dependencies - @...

Near line 847: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...@0.4.12 ## 0.4.4 ### Patch Changes - Updated dependencies - Updated dependencies [be5c9bc] - @ballerine/common@0.5.0 ...

Near line 881: Possible spelling mistake found.
Context: ...Patch Changes - 700d492: added missing changeset - fixed monorepo infra - Updated depend...

Near line 881: To make your text as clear as possible to all readers, do not use this foreign term. Possible alternatives are “below” or “further on” (in a document).
Context: ...dded missing changeset - fixed monorepo infra - Updated dependencies [700d492] - @b...

Near line 889: Possible spelling mistake found.
Context: ...tances of build with dist in @ballerine/workflow-browser-sdk ## 0.0.2 ### Patch Changes - Updated...

sdks/workflow-node-sdk/CHANGELOG.md (31)

Near line 1: Possible spelling mistake found.
Context: # @ballerine/workflow-node-sdk ## 0.6.7 ### Patch Changes - @baller...

Near line 32: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...e@0.6.4 ## 0.6.3 ### Patch Changes - Updated dependencies - Updated dependencies [0244d3e] - @ballerine/workflow-cor...

Near line 40: Possible spelling mistake found.
Context: ...e@0.6.3 ## 0.6.2 ### Patch Changes - bumo - Updated dependencies - @ballerine/w...

Near line 48: Possible spelling mistake found.
Context: ...e@0.6.2 ## 0.6.1 ### Patch Changes - ver bump - 9fe7a5c: bump - Updated dependencie...

Near line 50: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...Changes - ver bump - 9fe7a5c: bump - Updated dependencies - Updated dependencies [9fe7a5c] - @ballerine/workflow-cor...

Near line 138: Possible spelling mistake found.
Context: ...0.5.44 ## 0.5.43 ### Patch Changes - Ver bump - Updated dependencies - @ballerine/w...

Near line 272: Possible spelling mistake found.
Context: ... Updated dependencies - d5cca1e: added defintion validation - Updated dependencies [d5cc...

Near line 281: Possible spelling mistake found.
Context: ... ### Patch Changes - 8c888ce: Updated tsconfig and types - Updated dependencies [8c888...

Near line 313: Possible spelling mistake found.
Context: ...9 ## 0.5.18 ### Patch Changes - Bump ver - Updated dependencies - @ballerine/w...

Near line 341: Possible spelling mistake found.
Context: ...ch Changes - 801fc63: pre - 801fc63: Pre ver - 801fc63: Prerelease - 801fc63: bump...

Near line 348: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...: Pre bump - 801fc63: Bump - 801fc63: pre - Version bump - 801fc63: Pre release - 801fc63...

Near line 351: Possible spelling mistake found.
Context: ...ump - 801fc63: Pre release - 801fc63: Bumo - Updated dependencies [801fc63] - Upd...

Near line 361: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...39] - Updated dependencies [801fc63] - Updated dependencies - Updated dependencies [801fc63] - Updated dependencies [801f...

Near line 385: Possible spelling mistake found.
Context: ...ch Changes - b1cebf5: pre - b1cebf5: Pre ver - b1cebf5: Prerelease - b1cebf5: bump...

Near line 388: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...bf50: bump pre - b1cebf5: Version bump pre - Bump - b1cebf5: Pre bump - b1cebf5: Pre bu...

Near line 395: Possible spelling mistake found.
Context: ...pre - b1cebf5: Pre release - b1cebf5: Bumo - Updated dependencies [b1cebf5] - Upd...

Near line 401: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...50] - Updated dependencies [b1cebf5] - Updated dependencies - Updated dependencies [b1cebf5] - Updated dependencies [b1ce...

Near line 414: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...8.10 ### Patch Changes - Version bump pre - Updated dependencies - @ballerine/workflow-co...

Near line 430: Possible spelling mistake found.
Context: ...0.5.11-3e08f108.8 ### Patch Changes - Bumo - Updated dependencies - @ballerine/w...

Near line 438: Possible spelling mistake found.
Context: ...0.5.11-3e08f108.7 ### Patch Changes - Pre ver - Updated dependencies - @ballerine/w...

Near line 454: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...1-3e08f108.5 ### Patch Changes - bump pre - Updated dependencies - @ballerine/workflow-co...

Near line 470: This expression is normally spelled as one or with a hyphen.
Context: ...0.5.11-3e08f108.3 ### Patch Changes - Pre release - Updated dependencies - @ballerine/w...

Near line 478: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...0.5.11-3e08f108.2 ### Patch Changes - pre - Updated dependencies - @ballerine/workflow-co...

Near line 486: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...0.5.11-3e08f108.1 ### Patch Changes - pre - Updated dependencies - @ballerine/workflow-co...

Near line 628: Put a space after the comma.
Context: ... ### Patch Changes - added Autocomplete,DropdownInput,Multiselect,DatePicker,Checkbox,PhoneNu...

Near line 628: Put a space after the comma.
Context: ...nges - added Autocomplete,DropdownInput,Multiselect,DatePicker,Checkbox,PhoneNumber compone...

Near line 628: Put a space after the comma.
Context: ...d Autocomplete,DropdownInput,Multiselect,DatePicker,Checkbox,PhoneNumber components - Upgra...

Near line 628: Put a space after the comma.
Context: ...ete,DropdownInput,Multiselect,DatePicker,Checkbox,PhoneNumber components - Upgrade packag...

Near line 628: Possible spelling mistake found.
Context: ...wnInput,Multiselect,DatePicker,Checkbox,PhoneNumber components - Upgrade packages - Updated...

Near line 693: Possible spelling mistake found.
Context: ...f072188: reduced API drift and now node sdk resembles the browser sdk more closely ...

Near line 693: Possible spelling mistake found.
Context: ... and now node sdk resembles the browser sdk more closely - 668c125: fixed path to t...

services/workflows-service/CHANGELOG.md (26)

Near line 35: Possible spelling mistake found.
Context: ...5 ## 0.7.3 ### Patch Changes - Fixed invokedAt to be UTC epoch - Updated dependencies ...

Near line 44: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...k@0.6.4 ## 0.7.2 ### Patch Changes - Updated dependencies - Updated dependencies [0244d3e] - @ballerine/workflow-cor...

Near line 53: Possible spelling mistake found.
Context: ...k@0.6.3 ## 0.7.1 ### Patch Changes - bumo - Updated dependencies - @ballerine/w...

Near line 67: Possible spelling mistake found.
Context: ...- 9fe7a5c: Bump ### Patch Changes - ver bump - 9fe7a5c: bump alpha - 9fe7a5c: bu...

Near line 71: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...a5c10: bump - 9fe7a5c: Version bump - Updated dependencies - Updated dependencies [9fe7a5c] - Updated dependencies [9fe...

Near line 191: Possible spelling mistake found.
Context: ...0.5.44 ## 0.5.43 ### Patch Changes - Ver bump - Updated dependencies - @ballerine/c...

Near line 248: Possible spelling mistake found.
Context: ...### Patch Changes - Better logging for api plugin - Updated dependencies - @Ball...

Near line 430: Possible spelling mistake found.
Context: ...9 ## 0.5.18 ### Patch Changes - Bump ver - Updated dependencies - @ballerine/c...

Near line 458: Possible spelling mistake found.
Context: ...0.5.16 ## 0.5.15 ### Patch Changes - AsiaVerify integrations - Updated dependencies -...

Near line 469: Possible spelling mistake found.
Context: ...ch Changes - 801fc63: pre - 801fc63: Pre ver - 801fc63: Prerelease - 801fc63: bump...

Near line 476: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...: Pre bump - 801fc63: Bump - 801fc63: pre - Version bump - 801fc63: Pre release - 801fc63...

Near line 479: Possible spelling mistake found.
Context: ...ump - 801fc63: Pre release - 801fc63: Bumo - Updated dependencies [801fc63] - Upd...

Near line 489: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...39] - Updated dependencies [801fc63] - Updated dependencies - Updated dependencies [801fc63] - Updated dependencies [801f...

Near line 520: Possible spelling mistake found.
Context: ...ch Changes - b1cebf5: pre - b1cebf5: Pre ver - b1cebf5: Prerelease - b1cebf5: bump...

Near line 523: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...bf50: bump pre - b1cebf5: Version bump pre - Bump - b1cebf5: Pre bump - b1cebf5: Pre bu...

Near line 530: Possible spelling mistake found.
Context: ...pre - b1cebf5: Pre release - b1cebf5: Bumo - Updated dependencies [b1cebf5] - Upd...

Near line 536: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...50] - Updated dependencies [b1cebf5] - Updated dependencies - Updated dependencies [b1cebf5] - Updated dependencies [b1ce...

Near line 551: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...8.10 ### Patch Changes - Version bump pre - Updated dependencies - @ballerine/workflow-co...

Near line 571: Possible spelling mistake found.
Context: ...0.5.11-3e08f108.8 ### Patch Changes - Bumo - Updated dependencies - @ballerine/w...

Near line 581: Possible spelling mistake found.
Context: ...0.5.11-3e08f108.7 ### Patch Changes - Pre ver - Updated dependencies - @ballerine/w...

Near line 601: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...1-3e08f108.5 ### Patch Changes - bump pre - Updated dependencies - @ballerine/workflow-co...

Near line 621: This expression is normally spelled as one or with a hyphen.
Context: ...0.5.11-3e08f108.3 ### Patch Changes - Pre release - Updated dependencies - @ballerine/w...

Near line 631: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...0.5.11-3e08f108.2 ### Patch Changes - pre - Updated dependencies - @ballerine/workflow-co...

Near line 641: Prefixes followed by proper nouns or dates are typically hyphenated.
Context: ...0.5.11-3e08f108.1 ### Patch Changes - pre - Updated dependencies - @ballerine/workflow-co...

Near line 692: Possible typo detected.
Context: ... - Version bump - Updated dependencies [efedc9c] - Updated dependencies - @ballerine/...

Near line 960: This phrase is duplicated. You should probably use “Updated dependencies” only once.
Context: ...@0.5.1 ## 0.4.11 ### Patch Changes - Updated dependencies - Updated dependencies [be5c9bc] - @ballerine/common@0.5.0 ...

Additional comments not posted (44)

websites/docs/package.json (1)

20-20: Version update for @ballerine/common from ^0.9.3 to ^0.9.4 looks good.

Please ensure that the new version of @ballerine/common is compatible with the existing codebase by running all relevant tests.

examples/headless-example/package.json (2)

37-37: Version update for @ballerine/common from "0.9.3" to "0.9.4" looks good.

Please verify that the updated version of @ballerine/common integrates well with the headless example and does not introduce any regressions.

38-38: Version update for @ballerine/workflow-browser-sdk from "0.6.6" to "0.6.7" looks good.

Ensure that the new version of @ballerine/workflow-browser-sdk is fully compatible with the headless example by running comprehensive tests.

sdks/workflow-node-sdk/package.json (2)

4-4: Version update for @ballerine/workflow-node-sdk from "0.6.6" to "0.6.7" looks good.

Please confirm that the new version of the SDK functions as expected by conducting thorough testing.

31-31: Version update for @ballerine/workflow-core from "0.6.6" to "0.6.7" in the dependencies looks good.

Ensure that the updated version of @ballerine/workflow-core integrates seamlessly with the workflow-node-sdk by performing extensive tests.

sdks/workflow-browser-sdk/package.json (3)

4-4: Version update for @ballerine/workflow-browser-sdk from "0.6.6" to "0.6.7" looks good.

Please ensure that the new version of the SDK is fully functional by conducting comprehensive tests.

36-36: Version update for @ballerine/common from "0.9.3" to "0.9.4" in the dependencies looks good.

Verify that the updated version of @ballerine/common integrates well with the workflow-browser-sdk and does not introduce any regressions.

37-37: Version update for @ballerine/workflow-core from "0.6.6" to "0.6.7" in the dependencies looks good.

Ensure that the updated version of @ballerine/workflow-core integrates seamlessly with the workflow-browser-sdk by performing extensive tests.

packages/workflow-core/package.json (2)

4-4: Version update for @ballerine/workflow-core from "0.6.6" to "0.6.7" looks good.

Please confirm that the new version of the package functions as expected by conducting thorough testing.

34-34: Version update for @ballerine/common from "0.9.3" to "0.9.4" in the dependencies looks good.

Ensure that the updated version of @ballerine/common integrates seamlessly with the workflow-core package by performing extensive tests.

apps/kyb-app/package.json (3)

4-4: Version update for @ballerine/kyb-app from "0.3.6" to "0.3.7" looks good.

Please ensure that the new version of the app is fully functional by conducting comprehensive tests.

18-18: Version update for @ballerine/common from "^0.9.3" to "^0.9.4" in the dependencies looks good.

Verify that the updated version of @ballerine/common integrates well with the kyb-app and does not introduce any regressions.

20-20: Version update for @ballerine/workflow-browser-sdk from "0.6.6" to "0.6.7" in the dependencies looks good.

Ensure that the updated version of @ballerine/workflow-browser-sdk integrates seamlessly with the kyb-app by performing extensive tests.

sdks/web-ui-sdk/package.json (2)

24-24: Version update for @ballerine/web-ui-sdk from "1.5.4" to "1.5.5" looks good.

Please confirm that the new version of the SDK functions as expected by conducting thorough testing.

99-99: Version update for @ballerine/common from "0.9.3" to "0.9.4" in the dependencies looks good.

Ensure that the updated version of @ballerine/common integrates seamlessly with the web-ui-sdk by performing extensive tests.

apps/backoffice-v2/package.json (4)

3-3: Version update for @ballerine/backoffice-v2 from "0.7.5" to "0.7.6" looks good.

Please ensure that the new version of the backoffice app is fully functional by conducting comprehensive tests.

54-54: Version update for @ballerine/common from "0.9.3" to "0.9.4" in the dependencies looks good.

Verify that the updated version of @ballerine/common integrates well with the backoffice-v2 app and does not introduce any regressions.

56-56: Version update for @ballerine/workflow-browser-sdk from "0.6.6" to "0.6.7" in the dependencies looks good.

Ensure that the updated version of @ballerine/workflow-browser-sdk integrates seamlessly with the backoffice-v2 app by performing extensive tests.

57-57: Version update for @ballerine/workflow-node-sdk from "0.6.6" to "0.6.7" in the dependencies looks good.

Confirm that the updated version of @ballerine/workflow-node-sdk functions as expected within the backoffice-v2 app by conducting thorough testing.

services/workflows-service/package.json (2)

4-4: Version updated from 0.7.5 to 0.7.6.

48-50: Dependencies @ballerine/common, @ballerine/workflow-core, and @ballerine/workflow-node-sdk updated to new versions.

Ensure that these version updates are compatible with other parts of the system and do not introduce breaking changes.

packages/common/CHANGELOG.md (1)

3-8: Updated document schemas in version 0.9.4.

sdks/workflow-node-sdk/CHANGELOG.md (1)

3-8: Updated dependency @ballerine/workflow-core to version 0.6.7.

Ensure that this dependency update is tested thoroughly to avoid issues in production.

sdks/web-ui-sdk/CHANGELOG.md (1)

3-9: LGTM! The update to @ballerine/common@0.9.4 is correctly documented under version 1.5.5.

packages/workflow-core/CHANGELOG.md (1)

3-9: LGTM! The update to @ballerine/common@0.9.4 is correctly documented under version 0.6.7.

packages/common/src/schemas/documents/workflow/documents/schemas/UG.ts (6)

30-41: LGTM! The schema for 'certificate_of_registration' under 'proof_of_registration' is correctly defined with appropriate properties.

56-68: LGTM! The schema for 'trade_license' under 'proof_of_registration' is correctly defined with appropriate properties.

69-82: LGTM! The schema for 'trade_license' under 'proof_of_ownership' is correctly defined with appropriate properties.

96-108: LGTM! The schema for 'association_letter' under 'proof_of_registration' is correctly defined with appropriate properties.

125-138: LGTM! The schema for 'business_registration_form' under 'proof_of_ownership' is correctly defined with appropriate properties.

355-366: LGTM! The schema for 'property_rate' under 'proof_of_ownership' is correctly defined with appropriate properties.

examples/headless-example/CHANGELOG.md (1)

3-10: The changelog update for version 0.3.6 correctly documents the updated dependencies to @ballerine/common@0.9.4 and @ballerine/workflow-browser-sdk@0.6.7, aligning with the project-wide updates mentioned in the PR summary.

sdks/workflow-browser-sdk/CHANGELOG.md (1)

3-10: The changelog update for version 0.6.7 correctly documents the updated dependencies to @ballerine/common@0.9.4 and @ballerine/workflow-core@0.6.7, which is consistent with the cascading updates needed due to changes in core packages as described in the PR summary.

apps/kyb-app/CHANGELOG.md (1)

3-10: The changelog entry for version 0.3.7 correctly documents the updated dependencies. Ensure that the version numbers and package names are accurate and correspond to the actual updates made in the project dependencies.

services/workflows-service/CHANGELOG.md (1)

3-10: The CHANGELOG update correctly reflects the dependency updates as per the PR description. The formatting is consistent with previous entries.

apps/backoffice-v2/CHANGELOG.md (1)

7-10: Updated dependencies to @ballerine/common@0.9.4, @ballerine/workflow-browser-sdk@0.6.7, and @ballerine/workflow-node-sdk@0.6.7 are correctly documented for version 0.7.6.

packages/common/src/schemas/documents/workflow/documents/schemas/GH.ts (8)

29-41: Added a new schema for financial information under the type mtn_statement. Ensure that the properties like msisdn, accountHolderName, from, to, and timeRun are correctly validated and meet the business requirements.

112-147: Introduced a new schema for financial information under the type bank_statement. The schema includes a comprehensive list of banks and uses TypeStringEnum for issuer validation. Ensure that the printDate, accountHolderName, from, to, and accountNumber fields are appropriately used in the application logic.

725-735: New schema for proof of registration under the type certificate_of_registration includes fields like businessName, taxIdNumber, registrationNumber, and issueDate. Verify that these fields are required and correctly used in the business logic.

751-761: Added schema for proof of registration under the type operating_permit. This schema includes fields for businessName, registrationNumber, issueDate, and expirationDate. Confirm that the optional nature of registrationNumber and the inclusion of expirationDate align with business requirements.

777-787: Introduced a new schema for proof of registration under the type district_assembly_certificate. It includes fields like certificateNumber, businessName, registrationNumber, and issueDate. Ensure that these fields are utilized correctly in the application and meet the regulatory requirements.

808-822: New schema for proof of ownership under the type form_a includes detailed personal and business information. Verify the use of fields such as businessName, registrationNumber, taxIdNumber, issueDate, firstName, middleName, lastName, and dateOfBirth in the application's business logic.

836-844: Added schema for proof of ownership under the type receipt_for_permit. This schema includes businessName and issueDate. Confirm that these fields are sufficient for the intended use cases and comply with business and regulatory requirements.

859-868: Introduced a new schema for proof of ownership under the type property_rate. It includes businessName and payerName. Ensure that these fields are correctly utilized in the application and meet the necessary validation requirements.

Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

Share

Tips

Chat

There are 3 ways to chat with CodeRabbit:

Review comments: Directly reply to a review comment made by CodeRabbit. Example:
- I pushed a fix in commit <commit_id>.
- Generate unit testing code for this file.
- Open a follow-up GitHub issue for this discussion.
Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
- @coderabbitai generate unit testing code for this file.
- @coderabbitai modularize this function.
PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
- @coderabbitai generate interesting stats about this repository and render them as a table.
- @coderabbitai show all the console.log statements in this repository.
- @coderabbitai read src/utils.ts and generate unit testing code.
- @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (invoked as PR comments)

@coderabbitai pause to pause the reviews on a PR.
@coderabbitai resume to resume the paused reviews.
@coderabbitai review to trigger a review. This is useful when automatic reviews are disabled for the repository.
@coderabbitai resolve resolve all the CodeRabbit review comments.
@coderabbitai help to get help.

Additionally, you can add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.

CodeRabbit Configration File (`.coderabbit.yaml`)

You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
Please see the configuration documentation for more information.
If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

Visit our Documentation for detailed information on how to use CodeRabbit.
Join our Discord Community to get help, request features, and share feedback.
Follow us on X/Twitter for updates and announcements.

github-actions · 2024-04-22T07:52:51Z

PR Description updated to latest commit (ca4d26e)

github-actions · 2024-04-22T07:53:26Z

PR Review

⏱️ Estimated effort to review [1-5]	3, because the PR involves multiple files and significant changes to document schemas, which requires careful review to ensure data integrity and consistency across different document types and versions.
🧪 Relevant tests	No
🔍 Possible issues	Possible Bug: The `TypeStringEnum` function is used in the `propertiesSchema` for the 'bank_statement' type in both GH.ts and UG.ts files, but it is not defined or imported in the provided code. This might cause a runtime error if not handled properly.
🔍 Possible issues	Data Consistency: The `propertiesSchema` for various document types includes fields like `TypeNonEmptyString` and `TypeAlphanumericString` which are not standard TypeScript types or part of the `Type` object from 'io-ts' library. This needs clarification or correction to avoid potential type errors.
🔒 Security concerns	No

Code feedback:

relevant file	packages/common/src/schemas/documents/workflow/documents/schemas/GH.ts
suggestion	Consider adding a validation or a default value for the `issuer` field in the 'bank_statement' type to ensure data consistency and avoid potential runtime errors. [important]
relevant line	issuer: TypeStringEnum([

relevant file	packages/common/src/schemas/documents/workflow/documents/schemas/UG.ts
suggestion	It's recommended to handle the potential undefined function `TypeStringEnum` used in the 'trade_license' type. You might need to define this function or import it if it's part of an external library. [important]
relevant line	issuer: TypeStringEnum(['KCCA', 'Other']),

relevant file	packages/common/src/schemas/documents/workflow/documents/schemas/GH.ts
suggestion	Ensure that `TypeNonEmptyString` and `TypeAlphanumericString` are correctly defined or imported to prevent type errors in the 'bank_statement' and other document types. [important]
relevant line	accountHolderName: TypeNonEmptyString,

relevant file	packages/common/src/schemas/documents/workflow/documents/schemas/UG.ts
suggestion	Add error handling or default values for potentially undefined types like `TypeNonEmptyString` used in the 'mtn_statement' type to enhance robustness and maintainability of the code. [important]
relevant line	accountHolderName: TypeNonEmptyString,

✨ Review tool usage guide:

Overview:
The review tool scans the PR code changes, and generates a PR review which includes several types of feedbacks, such as possible PR issues, security threats and relevant test in the PR. More feedbacks can be added by configuring the tool.

The tool can be triggered automatically every time a new PR is opened, or can be invoked manually by commenting on any PR.

When commenting, to edit configurations related to the review tool (pr_reviewer section), use the following template:

/review --pr_reviewer.some_config1=... --pr_reviewer.some_config2=...

With a configuration file, use the following template:

[pr_reviewer]
some_config1=...
some_config2=...

See the review usage page for a comprehensive guide on using this tool.

github-actions · 2024-04-22T07:54:36Z

PR Code Suggestions

Category	Suggestions
Enhancement	Add validation to ensure `accountHolderName` is not empty. The `Type.String()` for `accountHolderName` should have a validation to ensure it is not empty. Consider adding a minimum length requirement or a pattern to ensure the field is not left blank. packages/common/src/schemas/documents/workflow/documents/schemas/GH.ts [37] -accountHolderName: Type.String(), +accountHolderName: Type.String({ minLength: 1 }),
Enhancement	Make `registrationNumber` a required field to enhance data integrity. The `Type.String()` method for `registrationNumber` in `propertiesSchema` should be made non-optional to enforce data integrity, assuming registration number is a critical attribute for a `certificate_of_registration`. packages/common/src/schemas/documents/workflow/documents/schemas/GH.ts [733] -registrationNumber: Type.Optional(TypeAlphanumericString), +registrationNumber: TypeAlphanumericString,
Best practice	Specify a format for `timeRun` to ensure data consistency. The `Type.String()` for `timeRun` in the `propertiesSchema` should specify a format if it represents a time or datetime value, to ensure correct and consistent data formatting. packages/common/src/schemas/documents/workflow/documents/schemas/GH.ts [40] -timeRun: Type.Optional(Type.String()), +timeRun: Type.Optional(Type.String({ format: 'date-time' })),
	Use a specific date type for date fields to enforce proper validation. For `Type.String()` fields representing dates such as `from` and `to`, it is recommended to use a more specific type like `Type.Date()` if available, to enforce proper date handling and validation. packages/common/src/schemas/documents/workflow/documents/schemas/GH.ts [144-145] -from: Type.String({ format: 'date' }), -to: Type.String({ format: 'date' }), +from: Type.Date(), +to: Type.Date(),
	Ensure version numbering follows semantic versioning principles. Ensure that the version number in the package.json file is updated in a consistent manner with semantic versioning principles. The minor version update from "0.6.6" to "0.6.7" should reflect backward-compatible enhancements or significant bug fixes. Verify that the changes in the dependencies and the overall functionality justify a minor version increment. packages/workflow-core/package.json [4] -"version": "0.6.7", +"version": "0.6.7", # Ensure this version change is justified as per semantic versioning
	Verify compatibility of updated dependencies. When updating dependencies, it's crucial to ensure that the specified versions do not introduce breaking changes or conflicts with other dependencies. The update from "@ballerine/common" version "0.9.3" to "0.9.4" should be checked for compatibility with other packages in the project. sdks/workflow-browser-sdk/package.json [36] -"@ballerine/common": "0.9.4", +"@ballerine/common": "0.9.4", # Verify compatibility with other dependencies
	Test system stability thoroughly after dependency updates. When updating multiple dependencies across different packages, consider the impact on the overall system and test extensively. The updates to "@ballerine/common", "@ballerine/workflow-core", and "@ballerine/workflow-node-sdk" should be thoroughly tested to ensure system stability. services/workflows-service/package.json [48-50] -"@ballerine/common": "0.9.4", +"@ballerine/common": "0.9.4", # Test updates thoroughly "@ballerine/workflow-core": "0.6.7", "@ballerine/workflow-node-sdk": "0.6.7",
	Ensure consistent dependency versioning for predictability. When updating a dependency version, ensure that the versioning is consistent (either using exact versions or compatible versions). The update to "@ballerine/common" uses a caret (^), which might introduce unexpected updates if not monitored. websites/docs/package.json [20] -"@ballerine/common": "^0.9.4", +"@ballerine/common": "0.9.4", # Consider using exact versions for better predictability
Bug	Remove the duplicate `issuer` field from `propertiesSchema`. The `issuer` field in the `propertiesSchema` for `bank_statement` is duplicated and should be removed as it is already defined at a higher level in the document object. packages/common/src/schemas/documents/workflow/documents/schemas/GH.ts [118-140] -issuer: TypeStringEnum([ - 'Absa Bank Ghana Limited', - 'Access Bank Ghana Plc', - ... -]), +// Removed duplicate issuer field
Maintainability	Review and update all dependencies when making changes. Ensure that all dependencies are updated to their latest stable versions if they are being changed to maintain compatibility and security. The update to "@ballerine/workflow-core" should be accompanied by a review of all dependencies for possible updates. sdks/workflow-node-sdk/package.json [31] -"@ballerine/workflow-core": "0.6.7", +"@ballerine/workflow-core": "0.6.7", # Review and update other dependencies as necessary

✨ Improve tool usage guide:

Overview:
The improve tool scans the PR code changes, and automatically generates suggestions for improving the PR code. The tool can be triggered automatically every time a new PR is opened, or can be invoked manually by commenting on a PR.

When commenting, to edit configurations related to the improve tool (pr_code_suggestions section), use the following template:

/improve --pr_code_suggestions.some_config1=... --pr_code_suggestions.some_config2=...

With a configuration file, use the following template:

[pr_code_suggestions]
some_config1=...
some_config2=...

See the improve usage page for a comprehensive guide on using this tool.

refactor(common): updated document schemas

ca4d26e

github-actions bot added enhancement New feature or request bug_fix labels Apr 22, 2024

github-actions bot added the Review effort [1-5]: 3 label Apr 22, 2024

tomer-shvadron approved these changes Apr 22, 2024

View reviewed changes

Omri-Levy merged commit 351fb9b into dev Apr 22, 2024
9 of 10 checks passed

Omri-Levy deleted the omri-levy/fix/support-old-schemas branch April 22, 2024 07:57

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

updated document schemas #2313

updated document schemas #2313

Omri-Levy commented Apr 22, 2024 •

edited by github-actions bot

changeset-bot bot commented Apr 22, 2024

coderabbitai bot commented Apr 22, 2024 •

edited

Chat

CodeRabbit Commands (invoked as PR comments)

CodeRabbit Configration File (`.coderabbit.yaml`)

Documentation and Community

github-actions bot commented Apr 22, 2024

github-actions bot commented Apr 22, 2024

github-actions bot commented Apr 22, 2024

updated document schemas #2313

updated document schemas #2313

Conversation

Omri-Levy commented Apr 22, 2024 • edited by github-actions bot

User description

Description

Related issues

Breaking changes

How these changes were tested

Examples and references

Checklist

Type

Description

Changes walkthrough

changeset-bot bot commented Apr 22, 2024

⚠️ No Changeset found

coderabbitai bot commented Apr 22, 2024 • edited

Walkthrough

Changes

Chat

CodeRabbit Commands (invoked as PR comments)

CodeRabbit Configration File (.coderabbit.yaml)

Documentation and Community

github-actions bot commented Apr 22, 2024

github-actions bot commented Apr 22, 2024

PR Review

github-actions bot commented Apr 22, 2024

PR Code Suggestions

Omri-Levy commented Apr 22, 2024 •

edited by github-actions bot

coderabbitai bot commented Apr 22, 2024 •

edited

CodeRabbit Configration File (`.coderabbit.yaml`)