-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Inventory Report layout #273
Comments
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
DWQ team is currently reviewing and we expect feedback shortly. |
This is the most recent iteration. uic-form-sample.pdf |
|
This is looking great! We had a few comments from our review:
|
Yes the email can be added. It's in there now and the spelling mistake is also corrected. Provided it's ok to add the footer... who decides if it's ok? |
@nathankota Do you know if it's ok for us to reuse the plss footer verbiage? If we're unsure, I can write something up. |
@rsparker-utah The disclaimer can be whatever you want/need it to be. I don't know of any use restrictions of the PLSS verbiage, but your team or legal representative (Cate?) would be better at answering whether it's appropriate for this particular app and data. You tell us what you want in there and we'll do it :) |
Ok, gotcha, that sounds good. Our review group seemed to like the plss verbiage, but I better run it by Cate first. Let's hold of on that piece for time being. I'll provide an update after I chat with Cate. |
I ran the disclaimer by our Data Management Specialist and have an update regarding the disclaimer. Could we add a sentence at the end of the existing disclaimer so it reads:
|
I've pushed the work I have done so far to dev. Please test it and see what you think. The appendix files are not currently being fetched and attached. Otherwise I think we have everything in the report that was in the mockup. Make sure your browser allows a popup. |
As far as report layout, it looks correct/accurate/great to me. @rsparker-utah we now defer to you for final validation. |
the well detail pdf's are now appended to the report. I deleted all of the cloud storage to remove any non pdf files. When you test you will need to update the well details for the inventory you are testing to add new files. |
I created a new test site (i.e., https://uic-inventory.dev.utah.gov/review/site/118/inventory/212), submitted an inventory, and downloaded the Inventory Report. I think the report layout looks great! I did have a couple review comments: 1.) On page 3 of the report, it looks like the "Construction" field isn't populating in the table under "Well Information". |
@rsparker-utah please give this another try |
I did a new round of testing on a new site (i.e., https://uic-inventory.dev.utah.gov/review/site/119/inventory/213), submitted an inventory, and downloaded the Inventory Report. I didn't come across any issues and I think the report layout looks really good! Thanks Steve! |
Using the mockup document as a guide (page 2 of the doc, onward).
(Side note: The PLSS app's monument record sheet was the inspiration for the report layout and design.)
First Page(s)
New page
New page
New page
New page
New page
Last page(s)
The text was updated successfully, but these errors were encountered: