Skip to content

PRXT - feat: lazy load tweet embeds#1411

Closed
punk6529 wants to merge 1 commit intomainfrom
codex/replace-static-import-with-dynamic-loading
Closed

PRXT - feat: lazy load tweet embeds#1411
punk6529 wants to merge 1 commit intomainfrom
codex/replace-static-import-with-dynamic-loading

Conversation

@punk6529
Copy link
Copy Markdown
Contributor

Summary

  • load the Tweet embed dynamically in DropPartMarkdown with a loading skeleton fallback to avoid bundling react-tweet eagerly
  • update the DropPartMarkdown test harness to mock next/dynamic, ensure prior behaviours continue to work, and cover the lazy-loading behaviour

Testing

  • npm run lint (passes with existing warnings in unrelated files)
  • npm run type-check (fails: project-wide test typings currently error in many pre-existing suites)
  • npm run test (fails: numerous suites abort because BASE_ENDPOINT isn’t set to an allowlisted domain in this environment)

https://chatgpt.com/codex/tasks/task_e_68c94060e4c8832197e27b2682a686da

Signed-off-by: OpenAI Assistant <123456+openaiassistant@users.noreply.github.com>
@sonarqubecloud
Copy link
Copy Markdown

@prxt6529 prxt6529 changed the title feat: lazy load tweet embeds PRXT - feat: lazy load tweet embeds Sep 30, 2025
@prxt6529
Copy link
Copy Markdown
Collaborator

Will be done in #1497

@prxt6529 prxt6529 closed this Sep 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants