-
Notifications
You must be signed in to change notification settings - Fork 358
[Maintenance] Review use of JSON Flattener #2930
Copy link
Copy link
Closed
Labels
enhancementNew feature or requestNew feature or requestgood first issueThese are recommended starting points for newcomers looking to make their first contributions.These are recommended starting points for newcomers looking to make their first contributions.triagedIssues labeled as 'Triaged' have been reviewed and are deemed actionable.Issues labeled as 'Triaged' have been reviewed and are deemed actionable.
Description
Is your feature request related to a problem?
During his review of #2926, @cwperks brought up a good point about potentially removing the dependency on the JSON-flattener.
Craig mentioned: "If that is all we use the json-flattener for than I would be in favor of less dependencies. If I am understanding it correctly, it is recursively going through a JSON structure and flattening it so all.entries.are.dot.separated and there is no object nesting?"
I am making this issue to unblock the original JSON bump PR but make sure that Craig's point is not forgotten.
To resolve this issue, we will want to review the use of the library and see whether it can be removed.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or requestgood first issueThese are recommended starting points for newcomers looking to make their first contributions.These are recommended starting points for newcomers looking to make their first contributions.triagedIssues labeled as 'Triaged' have been reviewed and are deemed actionable.Issues labeled as 'Triaged' have been reviewed and are deemed actionable.