Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Legal Names | Length Changes and Special Characters | Portal Registration Frontend | Portal Backend #360

Closed
4 tasks
typecastcloud opened this issue Jul 12, 2024 · 3 comments · Fixed by eclipse-tractusx/portal-backend#1043
Assignees
Labels
enhancement New feature or request

Comments

@typecastcloud
Copy link
Contributor

typecastcloud commented Jul 12, 2024

Description

We have some limitations to company REGEX that prevents onboarding of some customers.

1. Special Language Characters (Pending decision)

Currently the REGEX for legal company names is very restrictive. Special characters for some languages like German or French is allowed (e.g. üäöáè).
Other languages are not considered:

  • Polish: ĄĆĘŁŃÓŚŹŻ
  • Chinese: 你好 世界

Options:

  1. Keep as is: only A-Z a-z À-ÿ
  2. Allow characters for languages that are on white/yellow list
  3. Allow characters for all languages
Additional Information

For 1. companies need translated legal name to onboard their company.
Options 2. & 3. have some additional technical risks since some components (bpdm/clearing house) might not support special characters of this type.

2. Company Name Length (Required change for us)

There are companies with very long company names. We could find suppliers/partners in BPDM pool with up to 160 characters (more can happen but is unrealistic).
Now: 2-50 characters (currently counting white spaces)
Expected: 1-160

Additional Information

UI risk:

  • Very long legal name needs to work with input fields
  • Needs to work in company data

Compatibility risk ???

Acceptance Criteria

  • 1. Decision on special language characters is made
  • 2. If required by 1.: New REGEX allows for special language characters (Portal Registration Frontend & Portal Backend)
  • 3.1 New length for company names is applied to REGEX (Portal Registration Frontend & Portal Backend)
  • 3.2 UI works as intended

Additional Information

I have code ready for both changes with test cases. Waiting for decision on special character topic before creating PRs.

@jjeroch @evegufy

Usage of UNICODE scripts for extended REGEX like /p{L} requires minimum compile target of EC6
cc: @oyo

@typecastcloud typecastcloud added the enhancement New feature or request label Jul 12, 2024
typecastcloud added a commit to Cofinity-X/entry-portal-frontend-registration that referenced this issue Jul 12, 2024
Extended pattern with additional currency characters and length adjustment. Testing added.

Refs: eclipse-tractusx/portal#360
@jjeroch jjeroch added this to the Release 24.12 milestone Jul 16, 2024
typecastcloud added a commit to typecastcloud/portal-backend that referenced this issue Jul 17, 2024
…-end

Length increase is required for many companies. There are also companies with special characters at the start. Back-end was already misaligned with frontend-registration.

Refs: eclipse-tractusx/portal#360
typecastcloud added a commit to Cofinity-X/entry-portal-backend that referenced this issue Jul 23, 2024
…-end

Length increase is required for many companies. There are also companies with special characters at the start. Back-end was already misaligned with frontend-registration.

Refs: eclipse-tractusx/portal#360
typecastcloud added a commit to Cofinity-X/entry-portal-backend that referenced this issue Jul 23, 2024
…-end

Length increase is required for many companies. There are also companies with special characters at the start. Back-end was already misaligned with frontend-registration.

Refs: eclipse-tractusx/portal#360
dhiren-singh-007 pushed a commit to Cofinity-X/entry-portal-backend that referenced this issue Jul 24, 2024
…-end

Length increase is required for many companies. There are also companies with special characters at the start. Back-end was already misaligned with frontend-registration.

Refs: eclipse-tractusx/portal#360
dhiren-singh-007 pushed a commit to Cofinity-X/entry-portal-backend that referenced this issue Aug 1, 2024
…-end

Length increase is required for many companies. There are also companies with special characters at the start. Back-end was already misaligned with frontend-registration.

Refs: eclipse-tractusx/portal#360
dhiren-singh-007 pushed a commit to Cofinity-X/entry-portal-backend that referenced this issue Aug 5, 2024
…-end

Length increase is required for many companies. There are also companies with special characters at the start. Back-end was already misaligned with frontend-registration.

Refs: eclipse-tractusx/portal#360
dhiren-singh-007 pushed a commit to Cofinity-X/entry-portal-backend that referenced this issue Aug 8, 2024
…-end

Length increase is required for many companies. There are also companies with special characters at the start. Back-end was already misaligned with frontend-registration.

Refs: eclipse-tractusx/portal#360
@nicoprow
Copy link

BPDM Compatibility:

BPDM allows up to 250 characters for company names and is unicode compatible. Characters for Polish or Chinese for example are therefore also allowed.

I don't see any issue with this change regarding BPDM compatibility.

evegufy pushed a commit to eclipse-tractusx/portal-frontend-registration that referenced this issue Aug 14, 2024
* feat(legal entity): extended legal entity length

Extended pattern with additional currency characters and length adjustment. Testing added.

Refs: eclipse-tractusx/portal#360
ntruchsess pushed a commit to eclipse-tractusx/portal-backend that referenced this issue Aug 14, 2024
…-end

Length increase is required for many companies. There are also companies with special characters at the start. Back-end was already misaligned with frontend-registration.

Refs: eclipse-tractusx/portal#360
evegufy pushed a commit to eclipse-tractusx/portal-backend that referenced this issue Aug 14, 2024
* feat(company names): Increase company name length to align with front-end

Length increase is required for many companies. There are also companies with special characters at the start. Back-end was already misaligned with frontend-registration.

Refs: eclipse-tractusx/portal#360

---------

Co-authored-by: Dhirender Singh <[email protected]>
Co-authored-by: Dhirender Singh (Cofinity-X) <[email protected]>
Co-authored-by: Phil Schneider <[email protected]>
@typecastcloud
Copy link
Contributor Author

@MaximilianHauer
Hi Max, we still have chinese/thai/polish characters open as a topic.

There is a way to simplify REGEX via unicode scripts. \p{Letter} covers all letters in any language for example. See unicode scripts.

This would require our frontends to compile at least ES6 since support from then on. (compilerOptions still targets es5)

CC: @ybidois

@oyo
Copy link
Contributor

oyo commented Aug 19, 2024

From my pov okay. Support of ES6 is quite good globally (roughly 1 percent less than ES5) and enterprise users are more likely to run an up-to-date browser: https://caniuse.com/?search=es6 vs. https://caniuse.com/?search=es5

typecastcloud added a commit to Cofinity-X/entry-portal-frontend-registration that referenced this issue Aug 28, 2024
The legal entity pattern now supports and uses letter and currency unicode categories. This required update of ecma script to version 6.

Refs: eclipse-tractusx/portal#360
Phil91 added a commit to AnuragNagpure/portal-backend that referenced this issue Sep 6, 2024
* feat(company names): Increase company name length to align with front-end

Length increase is required for many companies. There are also companies with special characters at the start. Back-end was already misaligned with frontend-registration.

Refs: eclipse-tractusx/portal#360

---------

Co-authored-by: Dhirender Singh <[email protected]>
Co-authored-by: Dhirender Singh (Cofinity-X) <[email protected]>
Co-authored-by: Phil Schneider <[email protected]>
typecastcloud added a commit to Cofinity-X/entry-portal-frontend-registration that referenced this issue Sep 18, 2024
The legal entity pattern now supports and uses letter and currency unicode categories. This required update of ecma script to version 6.

Refs: eclipse-tractusx/portal#360
typecastcloud added a commit to Cofinity-X/entry-portal-backend that referenced this issue Sep 19, 2024
The Company validation expression in the ValidationExpressions class has been updated to support a wider range of characters, including special characters and non-Latin alphabets. This change ensures that the validation accurately captures company names from different languages and regions.

Fixes: eclipse-tractusx/portal#360
Refs: eclipse-tractusx/portal-frontend-registration#250
typecastcloud added a commit to Cofinity-X/entry-portal-frontend-registration that referenced this issue Sep 27, 2024
The legal entity pattern now supports and uses letter and currency unicode categories. This required update of ecma script to version 6.

Refs: eclipse-tractusx/portal#360
typecastcloud added a commit to Cofinity-X/entry-portal-backend that referenced this issue Sep 27, 2024
The Company validation expression in the ValidationExpressions class has been updated to support a wider range of characters, including special characters and non-Latin alphabets. This change ensures that the validation accurately captures company names from different languages and regions.

Fixes: eclipse-tractusx/portal#360
Refs: eclipse-tractusx/portal-frontend-registration#250
typecastcloud added a commit to Cofinity-X/entry-portal-frontend that referenced this issue Oct 1, 2024
The legal entity pattern now supports and uses letter and currency unicode categories. This required update of ecma script to version 6.

Refs: eclipse-tractusx/portal#360
typecastcloud added a commit to Cofinity-X/entry-portal-backend that referenced this issue Oct 11, 2024
The Company validation expression in the ValidationExpressions class has been updated to support a wider range of characters, including special characters and non-Latin alphabets. This change ensures that the validation accurately captures company names from different languages and regions.

Fixes: eclipse-tractusx/portal#360
Refs: eclipse-tractusx/portal-frontend-registration#250
typecastcloud added a commit to Cofinity-X/entry-portal-frontend that referenced this issue Oct 11, 2024
The legal entity pattern now supports and uses letter and currency unicode categories. This required update of ecma script to version 6.

Refs: eclipse-tractusx/portal#360
ntruchsess pushed a commit to eclipse-tractusx/portal-backend that referenced this issue Oct 14, 2024
The Company validation expression in the ValidationExpressions class has been updated to support a wider range of characters, including special characters and non-Latin alphabets. This change ensures that the validation accurately captures company names from different languages and regions.

Fixes: eclipse-tractusx/portal#360
Refs: eclipse-tractusx/portal-frontend-registration#250

* Adjust changelog
* Add remarks to Company regex
* Update packages
typecastcloud added a commit to Cofinity-X/entry-portal-frontend-registration that referenced this issue Oct 22, 2024
The legal entity pattern now supports and uses letter and currency unicode categories. This required update of ecma script to version 6.

Refs: eclipse-tractusx/portal#360
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants