-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
🚧 XRPC rework #3814
base: main
Are you sure you want to change the base?
🚧 XRPC rework #3814
Conversation
Your Render PR Server URL is https://social-app-pr-3814.onrender.com. Follow its progress at https://dashboard.render.com/web/srv-copnj8tjm4es73efeaqg. |
b72440f
to
7c9f2cd
Compare
|
7c9f2cd
to
ae00556
Compare
src/lib/link-meta/link-meta.ts
Outdated
const response = await fetch( | ||
`${LINK_META_PROXY(agent.service.toString() || '')}${encodeURIComponent( | ||
`${LINK_META_PROXY(serciceUrl.toString() || '')}${encodeURIComponent( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a way we could do this without having to have access to the .getServiceUrl
? This is the only place this method (from @atproto/api
) is needed for.
7187def
to
99d7966
Compare
ae00556
to
1a535b0
Compare
99d7966
to
7b15c8d
Compare
1a535b0
to
24bf6f8
Compare
24bf6f8
to
99f74f4
Compare
99f74f4
to
a053710
Compare
This adapts the social-app to the changes introduced by bluesky-social/atproto#2434
This PR should not be merged before those changes are published in an NPM package.