Skip to content
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

compose and style data dictionary HTML #1296

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

damonmcc
Copy link
Member

@damonmcc damonmcc commented Dec 2, 2024

related to #561

follow-up to #1269

These changes use the new approach for generating HTML/PDFs from metadata yml:

  • combine modular jinja templates
  • apply css styling to HTML before generating the PDF

These changes do not apply all of the formatting seen in the Figma designs. I got stuck with nesting html elements and getting the text to wrap correctly so I focused on creating/using modular jinja files for a data dictionary.

Tested changes by running Template DB export stage locally and looking at the PDF and here's the PDF generated by the PR Template DB build data_dictionary.pdf.

TODOs

  • download entire org metadata so that dataset + product + org fields are available during artifact generation
  • render all column fields
  • generate a PLUTO data dictionary pdf

Copy link

codecov bot commented Dec 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.67%. Comparing base (d162e08) to head (45db96d).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1296   +/-   ##
=======================================
  Coverage   69.67%   69.67%           
=======================================
  Files         111      111           
  Lines        5913     5913           
  Branches      659      659           
=======================================
  Hits         4120     4120           
  Misses       1661     1661           
  Partials      132      132           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@damonmcc damonmcc force-pushed the dm-metadata-pdf branch 4 times, most recently from 9b365ce to 762c33b Compare December 2, 2024 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: New
Development

Successfully merging this pull request may close these issues.

1 participant