While testing out the fix for #45 found that summaries did not show for all instances. Realized that we were using some <b> and <i> tags to make them "pretty" in Github. This 100% broke b/c .text on an lxml element will only grab the text inside the immediate element and not the child elements, so need to use itertext() to ensure we grab all of it.
Testing
Added a unit test as well as confirmed that the sections now render with summaries.
While testing out the fix for #45 found that summaries did not show for all instances. Realized that we were using some
<b>
and<i>
tags to make them "pretty" in Github. This 100% broke b/c.text
on an lxml element will only grab the text inside the immediate element and not the child elements, so need to useitertext()
to ensure we grab all of it.Testing
Added a unit test as well as confirmed that the sections now render with summaries.