cdisc-org / conformance-rules-editor

MIT License
6 stars 3 forks source link

Don't allow core id to be set manually #199

Closed gerrycampion closed 1 year ago

gerrycampion commented 1 year ago

This update does not allow an author to manually set Core.Id to a valid id.
If a rule is saved and the core id:

then the core id is replaced by empty string.

You can test this by creating new rules or modifying existing rules with valid and invalid core ids. A valid core id matches the pattern /^CORE-\d{6}$/

github-actions[bot] commented 1 year ago

Azure Static Web Apps: Your stage site is ready! Visit it here: https://icy-flower-095494b10-199.centralus.azurestaticapps.net

nhaydel commented 1 year ago

This issue is good to close.

Tested by manually assigning a valid rule id and having it overwritten to an empty string on save. Manually assigned an invalid rule id and it was not overwritten