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

Remove list/object parsing from mapByValue #175

Closed
dafreels opened this issue Oct 2, 2020 · 0 comments
Closed

Remove list/object parsing from mapByValue #175

dafreels opened this issue Oct 2, 2020 · 0 comments
Assignees
@dafreels
Labels
1.8 dev complete

Comments

@dafreels
Copy link
Contributor

dafreels commented Oct 2, 2020

PipelineStepMapper.mapByValue has old logic which will parse any string as a list or object where the first character of the string is [ or {. This logic was in place specifically to handle the libraries inability to consume objects and list from JSON. This is no longer needed and is causing issues with globals now that they may be expanded.
@dafreels dafreels added the 1.8 label Oct 2, 2020
@dafreels dafreels self-assigned this Jan 21, 2021
dafreels added a commit to dafreels/metalus that referenced this issue Jan 21, 2021
@dafreels
Acxiom#175 Removed code that would attempt to create a list/object by…
90f1c29 
… parsing string contents that start with [ or {.
@dafreels dafreels mentioned this issue May 21, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dafreels dafreels

Labels
1.8 dev complete
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@dafreels