Skip to content
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

docs: 'BskyAgent' is deprecated #272

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

adamclark64
Copy link

@adamclark64 adamclark64 commented Dec 2, 2024

WHAT: update get-started.mdx docs

WHY:

'BskyAgent' is deprecated.ts(6385)
bsky-agent.d.ts(2, 5): The declaration was marked as deprecated here.

@bamnet
Copy link

bamnet commented Jan 19, 2025

AtpAgent is planned to be deprecated soon too. The inline docs recommend jumping straight to Agent / CredentialSession like:

const session = new CredentialSession(new URL("https://api.bsky.app"));
  
export const agent = new Agent(session);

(regardless, this feels like one of the more impactful PRs to merge since it's in the getting started flow)

@adamclark64
Copy link
Author

AtpAgent is planned to be deprecated soon too. The inline docs recommend jumping straight to Agent / CredentialSession like:

const session = new CredentialSession(new URL("https://api.bsky.app"));
  
export const agent = new Agent(session);

happy to make this change if it makes more sense

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.

2 participants