fullPage: true? #9

Closed
Abcmsaj wants to merge 1 commits from patch-1 into master
Abcmsaj commented 2020-09-01 18:14:38 -05:00 (Migrated from github.com)

Unsure if you were aware but passing fullPage: true into the page.screenshot command will return the whole page. Really useful for some websites like Wikipedia where you may want more info. Perhaps you could add it as an optional arg to the command?

Unsure if you were aware but passing fullPage: true into the page.screenshot command will return the whole page. Really useful for some websites like Wikipedia where you may want more info. Perhaps you could add it as an optional arg to the command?
Abcmsaj commented 2020-09-01 18:15:44 -05:00 (Migrated from github.com)

Really cool project btw. I've spent the last week making my own Discord.js bot for a Fantasy Football group and had no idea I could integrate with Puppeteer to post screenshots to the channel - you've saved me a ton of time, thank you!

Really cool project btw. I've spent the last week making my own Discord.js bot for a Fantasy Football group and had no idea I could integrate with Puppeteer to post screenshots to the channel - you've saved me a ton of time, thank you!

good idea; I should add a flag parser; I would also like a flag to specify viewport

good idea; I should add a flag parser; I would also like a flag to specify viewport

closed with 71fe7a19d7

closed with 71fe7a19d736206b3843404926b1d230ea014f22

Pull request closed

Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: lamp/puppeteer-discord-bot#9
There is no content yet.