In the first of two very nit-picky style bug reports...
While the if statements introduced on lines 78 onward in 44f01dce7f6cdab5a2c6972e397b357dedae6ef7's data_formatters.js are correct, the file holds an index for those strategies, effectively acting like a factory. While we currently only support two formats, that list may expand in the future and this approach provides a community-standard for graceful growth. Either way, that index (serializationStrategies) should be removed or the if statements replaced.
In the first of two very nit-picky style bug reports...
While the if statements introduced on lines 78 onward in 44f01dce7f6cdab5a2c6972e397b357dedae6ef7's data_formatters.js are correct, the file holds an index for those strategies, effectively acting like a factory. While we currently only support two formats, that list may expand in the future and this approach provides a community-standard for graceful growth. Either way, that index (serializationStrategies) should be removed or the if statements replaced.