jpetso / formstamper Goto Github PK

Consider using Bootstrap for the modal

The spread operator works for arrays but not for objects. The pattern we're currently using instead is Object.assign(), which is necessary to avoid mutations. The issue and the solution has been documented by others although I don't know what the best solution might be (1, 2, 3, 4, 5, 6).

Program will not crash if there is no PDFtk binary, but no PDFs will be generated, which is a major bug!

VLAvailableFields is largely a layer containing layout logic and for mapping VLFieldMappings. The logic could be moved into App, and delete VLAvailableFields.

Currently, there is a UX bug that is non-breaking.See more here.

See pdf.worker.js.

Known errors so far:
-PDF generation

Jest provides serialized snapshot testing and the ability to write assertions, which is useful especially in place of loading PDFs and CSVs.

-provide instructions for the end-user so that they can install and operate the software
-provide an overview for developers who wish to customize the software by:
-describing the stack
-describing the dataflow
-describing specific functionality

After loading a PDF, then loading a CSV, and mapping at least one CSV column to one PDF field, the fieldMappings array is no longer empty. If you then reload a PDF, retain the mappings for PDF fields that are in both the previously loaded PDF file and the newly loaded PDF file. Meanwhile, when reloading a CSV, retain the mappings for CSV column headers that are in both the previously loaded CSV file and the newly loaded CSV file.

In the case where the user does not want to retain mappings after creating a batch of PDFs, there needs to be a reset button to clear fieldMappings.

It is an industry standard and will help with consistent formatting style.