scientist-softserv / palni_palci_knapsack

Apache License 2.0
1 stars 0 forks source link

:unicorn: Hide PDF inputs from edit form #57

Closed ShanaLMoore closed 1 week ago

ShanaLMoore commented 3 weeks ago

Verify that PDF.js is working on pals knapsack

UPDATE: Original title: PDF.js not functional

TODO: PDF.js should be check boxes on the edit form's files tab only. Hide the input field boxes from the description tab of the edit form. PS. Marking the metadata yaml properties with form > display: false did not hide them.

ShanaLMoore commented 3 weeks ago

Tested on STAGING

PS. The iiif print code to split PDFs has not been merged at this point.

Prerequisites:

Image

When PDF.js checkbox is off: ✅

![Image](https://github.com/scientist-softserv/palni_palci_knapsack/assets/10081604/de209fa2-318b-406c-a65b-35a78a6c81da) ![Image](https://github.com/scientist-softserv/palni_palci_knapsack/assets/10081604/8bc37479-cc9c-4eed-adc9-d0ac94f889fc)

When PDF.js checkbox is on: ✅

![Image](https://github.com/scientist-softserv/palni_palci_knapsack/assets/10081604/54ab8e35-eea5-4da4-b76e-54aec2d7a8fc) ![Image](https://github.com/scientist-softserv/palni_palci_knapsack/assets/10081604/c57c7fce-4a79-4b04-b972-27202e4eb6d3)
ShanaLMoore commented 3 weeks ago

Show PDF options should NOT be on the form as inputs: :x:

EDIT > FILES TAB ✅

Image

FORM :x:

Image

ShanaLMoore commented 3 weeks ago

Locally, I do not see these check boxes in the files tab 🤔 but in pals staging I do.

UPDATE: RESOLVED. I didn't have my worker container running so it didn't know about the uploaded file.

ShanaLMoore commented 3 weeks ago

Setting down due to a shift in priorities.

TODO: Hide the input fields from the description tab

ShanaLMoore commented 1 week ago

dupe #70