Skip to content

PyPi docs out of date #77

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
filip-michalsky opened this issue Dec 4, 2024 · 1 comment · Fixed by #80
Closed

PyPi docs out of date #77

filip-michalsky opened this issue Dec 4, 2024 · 1 comment · Fixed by #80

Comments

@filip-michalsky
Copy link

https://pypi.org/project/browserbase/

looking at the start of the README:

`import os
from playwright.sync_api import Playwright, sync_playwright
from browserbase import Browserbase

BROWSERBASE_API_KEY = os.environ.get("BROWSERBASE_API_KEY")
BROWSERBASE_PROJECT_ID = os.environ.get("BROWSERBASE_PROJECT_ID")

bb = Browserbase(
# This is the default and can be omitted
api_key=BROWSERBASE_API_KEY,
)

session = client.sessions.create(
project_id=BROWSERBASE_PROJECT_ID,
)
print(session.id)`

does not work since i think the newer version of the python SDK for browserbase does not pass the env vars explicitly. Also, the part of readm above is defining bb as the client but then using variable client to create a session which I think is a typo?

@kamath kamath linked a pull request Dec 4, 2024 that will close this issue
@kamath
Copy link
Member

kamath commented Dec 4, 2024

Hey @filip-michalsky thanks for bringing this to our attention! We're resolving this in the linked PR, #80.

@kamath kamath closed this as completed in #80 Dec 4, 2024
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 a pull request may close this issue.

2 participants