Skip to content

Handle dynamic form submission and allow wkhtmltopdf path override#13

Open
mikerobinsonshow wants to merge 2 commits into
mainfrom
codex/update-pdf_dir-to-absolute-path-dur8c2
Open

Handle dynamic form submission and allow wkhtmltopdf path override#13
mikerobinsonshow wants to merge 2 commits into
mainfrom
codex/update-pdf_dir-to-absolute-path-dur8c2

Conversation

@mikerobinsonshow
Copy link
Copy Markdown
Owner

Summary

  • send dynamic form data to the Flask backend via a new handleSubmit handler
  • display PDF download link or server-side validation errors
  • add placeholder in the template for result messages
  • ensure server only validates submitted fields
  • add unit tests for server-side validation
  • gracefully handle missing wkhtmltopdf and return an error instead of crashing
  • allow setting a manual wkhtmltopdf path via the WKHTMLTOPDF_CMD environment variable
  • add unit test verifying that a custom wkhtmltopdf path is used

Testing

  • python -m py_compile app.py
  • python -m pytest

https://chatgpt.com/codex/tasks/task_e_68b4a864d314832fabcb7d16b3e9205c

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant