Skip to content

initial preact implementation#19

Open
jacob-ebey wants to merge 10 commits intomainfrom
preact
Open

initial preact implementation#19
jacob-ebey wants to merge 10 commits intomainfrom
preact

Conversation

@jacob-ebey
Copy link
Copy Markdown
Collaborator

No description provided.

@brophdawg11 brophdawg11 self-assigned this Sep 8, 2022
<Suspense
fallback={<p id="lazy-error">Loading error...</p>}
key={`error-${location.key}`}
>
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jacob-ebey So this is a total hack but solves the issue for the time being 🤷‍♂️. I made an issue in the Projects view to look into solving the never-resolving promise issue to unblock this

@brophdawg11
Copy link
Copy Markdown
Owner

Unsure what's going on with the CI tests - they seem ok locally but preact tests are flakey in CI. Will look into it more another time

@IshanKBG
Copy link
Copy Markdown

IshanKBG commented Jan 21, 2023

Any update on this PR?

Really excited to see preact support

@brophdawg11
Copy link
Copy Markdown
Owner

@IshanKBG I believe we were waiting on a suspense fix in preact so defer would work right, but there's probably no reason that needs to hold up an initial merge here. I'll see if I can find some time this week or next to maybe remove defer/Await from this initial PR and get a working version out and then we can focus on those in a separate PR

@IshanKBG
Copy link
Copy Markdown

IshanKBG commented Jan 25, 2023

@brophdawg11
Oh nice
Thanks for notifying me

@nicksrandall
Copy link
Copy Markdown

I'd love to use this if possible!

@brophdawg11 brophdawg11 removed their assignment Jul 16, 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 this pull request may close these issues.

4 participants