Skip to content
This repository has been archived by the owner on Aug 22, 2023. It is now read-only.

Added new method, view(), for use when debugging #129

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Added new method, view(), for use when debugging #129

wants to merge 3 commits into from

Conversation

BritishWerewolf
Copy link
Contributor

I have added in a new method that can be called in the same way that stream is in order to print raw HTML to the browser.
This is useful for debugging the HTML to view what the code will look like - roughly.

Also, I have modified the comments on the config array so that they reflect the wording from the Mpdf documentation.

I have added in a new method that can be called in the same way that `stream` is in order to print raw HTML to the browser.  
This is useful for debugging the HTML to view what the code will look like - roughly.

Also, I have modified the comments on the config array so that they reflect the wording from the Mpdf documentation.
Copy link
Contributor Author

@BritishWerewolf BritishWerewolf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Accidentally added in some extra options.

@verbeeksteven
Copy link

@niklasravnsborg - Need this for testing, could we get it merged?

@bomshteyn
Copy link

This seems like a very good idea, thanks @JustCarty.

@niklasravnsborg is this being merged in?

@niklasravnsborg
Copy link
Owner

Hey guys! I hear you on this. I cannot merge at the moment. Maybe @erfansahaf can look at it? Thanks for your patience. :) Have a good weekend.

@erfansahaf
Copy link
Contributor

@niklasravnsborg I'm busy for now, but I'll review all merge requests as soon as I can.

Base automatically changed from master to main March 4, 2021 12:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants