-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Better starterpack embed #4659
Better starterpack embed #4659
Conversation
Your Render PR Server URL is https://social-app-pr-4659.onrender.com. Follow its progress at https://dashboard.render.com/web/srv-cpu9ahiju9rs73fbp7dg. |
|
|
||
export async function resolveShortLink(shortLink: string) { | ||
const controller = new AbortController() | ||
const to = setTimeout(() => controller.abort(), 2e3) |
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.
these should resolve in two seconds, but we can bump to maybe 5 if we need to.
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.
Tested on iOS simulator, seems solid
* origin/main: bskyogcard: support emoji, more languages, long starter pack names (#4668) Run intl extract Add some events to landing screen (#4664) update follows when pressing follow all (#4663) Better starterpack embed (#4659) [Starter Packs] Posts tab (#4660) Run intl extract [D1X] Onboarding interest display names (#4657) Mark two starter pack strings for localization (#4655) Refactor nested conditions in the starter pack wizard (#4652)
without whitespace
Temporarily using an external embed for this. The app view is actually returning a
NotFound
record for these, so the normal way of embedding a record won't do great. The image being displayed keeps the info fresh though.Will add go.bsky.app resolution in a separate PR.
Note: resolution won't work on localhost, because of the origin header. Has to be on a bsky.app domain.