Skip to content

Centralization of semicolon-delimited value handling for the multi-value control by introducing a shared codec utility#399

Open
tbernard-dennemeyer wants to merge 4 commits into
microsoft:masterfrom
tbernard-dennemeyer:master
Open

Centralization of semicolon-delimited value handling for the multi-value control by introducing a shared codec utility#399
tbernard-dennemeyer wants to merge 4 commits into
microsoft:masterfrom
tbernard-dennemeyer:master

Conversation

@tbernard-dennemeyer
Copy link
Copy Markdown

@tbernard-dennemeyer tbernard-dennemeyer commented Apr 17, 2026

Dear,

This change refactors multi-value field handling to use a single shared codec so parsing and serialization are consistent across the extension.

Changed files

[valueCodec.ts]
Added shared parsing logic for semicolon-delimited values.
Added shared serialization logic that trims entries, removes empty values, and joins with semicolons.
Added compatibility helper for allowed-values behavior.
[MultiValueEvents.tsx]
Replaced inline split/trim parsing with shared codec parsing.
Replaced inline join serialization with shared codec serialization.
[getSuggestedValues.ts]
Replaced inline split/trim parsing of configured Values input with shared codec parsing.

Sample values

Input selections:
Customer A
Customer B
Serialized value:
Customer A;Customer B
Parsed result:
Customer A
Customer B

Outcome

One canonical implementation for value parsing/serialization.
Cleaner maintenance and fewer inconsistencies between read/write paths.

@tbernard-dennemeyer
Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree [company="Dennemeyer"]

@tbernard-dennemeyer
Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree company="Dennemeyer"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant