Skip to content

Fix call signature for local screenshot#108

Closed
conorbranagan wants to merge 1 commit intobrowserbase:mainfrom
conorbranagan:conor/fix-local-screenshot
Closed

Fix call signature for local screenshot#108
conorbranagan wants to merge 1 commit intobrowserbase:mainfrom
conorbranagan:conor/fix-local-screenshot

Conversation

@conorbranagan
Copy link
Copy Markdown

why

This function takes a set of kwargs rather than a dictionary, the call was failing.

what changed

  • Pass through the payload as kwargs
  • Added a test for this path
  • Some whitespace was changed on save (I can remove this if this is an issue)

test plan

Added a new test for this and verified this locally in the work I'm doing using this library.

This function takes a set of kwargs rather than a dictionary.
@conorbranagan
Copy link
Copy Markdown
Author

Ah! I had missed #106, will close.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant