quarto get_chapters() compatibility

[cansavvy]

Purpose/implementation Section

What changes are being implemented in this Pull Request?

The get_chapters() function is used to collect chapter information. This is used in turn by the bookdown_to_leanpub() function in ottrpal.

It retrieves chapters based on the html node structure for bookdown but quarto has a different structure.

So this is adding compatibility for the quarto structure.

What this needs

The only true changes are in R/bookdown_to_leanpub.R the rest is styling changes.

@howardbaek Can you double check this and see if it makes sense?

[howardbaik]

I ran get_chapters() with index.html generated by the Quarto version of ottr, and got this error message:

Error in `dplyr::mutate()`:
ℹ In argument: `url = paste0(base_url, data_path)`.
Caused by error:
! object 'data_path' not found

The error stems from Line 472.

[cansavvy]

We pair programmed on this and determined it was a simple logic issue.

Can you test this again?

[howardbaik]

After applying my code suggestion, I get this data frame for chapt_data:

> chapt_data
                chapt_titles data_level                       data_path
1 Introduction to the Course          1                      intro.html
2                New Chapter          2  first-section-new-chapter.html
3                New Chapter          3 second-section-new-chapter.html
4                New Chapter          4  third-section-new-chapter.html
5                 Conclusion          5                 conclusion.html

[cansavvy]

Thanks for these fixes @howardbaek I think this all makes sense.