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.

Sign up for GitHub

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

Jump to bottom

MiniLcm remove empty MultiString entries automatically #1292

Open
hahn-kev opened this issue Nov 29, 2024 · 0 comments
Open

MiniLcm remove empty MultiString entries automatically #1292

hahn-kev opened this issue Nov 29, 2024 · 0 comments
Assignees
@rmunn
Labels
📖 MiniLcm issues related to miniLcm library code, includes fwdat bridge and lcmCrdt

Comments

@hahn-kev
Copy link
Collaborator

Describe the feature
right now our validation of MultiStrings prevent having empty string values, this adds friction on the API caller, it would be nicer to just remove those empty strings so they don't exist. This could be done at the json serialization layer.

Who is this feature for?
MiniLcm api devs
@hahn-kev hahn-kev added the 📖 MiniLcm issues related to miniLcm library code, includes fwdat bridge and lcmCrdt label Nov 29, 2024
@hahn-kev hahn-kev mentioned this issue Nov 29, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rmunn rmunn

Labels
📖 MiniLcm issues related to miniLcm library code, includes fwdat bridge and lcmCrdt
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rmunn @hahn-kev