Skip to content

Conversation

@Chigala
Copy link
Contributor

@Chigala Chigala commented Sep 11, 2023

Task 1 - Sveltekit Adapter

  • Created a new folder in packages called sveltekit.
  • Created a basic Sveltekit project that uses the adapter. Place it in the examples/sveltekit-example folder.
  • Included a basic Job in the Sveltekit project
  • create a @trigger.dev/svelte package
  • Provide a screenshot of your terminal and the dashboard showing that the Adapter is working.
Screenshot 2023-09-11 at 7 09 48 AM Screenshot 2023-09-11 at 7 10 07 AM Screenshot 2023-09-11 at 7 10 23 AM

Task 2 - Add a README.md file

  • Created a new README.md file inside the packages/sveltekit folder.

Task 3 - Write the documentation

  • Added the sveltekit. manual-setup-guide manual-setup-sveltekit.mdx file in /docs/_snippets/.
  • Updated the table at the bottom of the client-adaptors.mdx page with the Sveltekit framework.

Task 4 - Update the webapp onboarding

  • In the route.tsx page located in _app.orgs.$organizationSlug.projects.$projectParam.setup.sveltekit, replaced the <FrameworkComingSoon/> component with new onboarding steps.
  • In the FrameworkSelector.tsx file, add "supported" to the framework you’ve worked on.

I didn't include trigger init CLI support for the framework because I don't know what the Trigger team has in mind concerning how to handle initialization for the other separate frameworks. If this is needed I don't mind implementing it. I'll also love your feedback on this!

/claim #453
/closes #453

@changeset-bot
Copy link

changeset-bot bot commented Sep 11, 2023

⚠️ No Changeset found

Latest commit: 7391ad9

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@matt-aitken
Copy link
Member

@Chigala I made some changes to the Remix PR before merging it.

  • The blank project page in the app was broken – there was a button with no text so it was just a rectangle. I also improved the text a bit.
  • I made some of the docs a bit more specific
  • Some tsconfig paths were missing
  • "next" was a dependency of the remix package
  • A specific version of Remix was in the main dependencies of the package. This would have caused issues with users who were on different versions. I moved it do be a devDependency only.

Can you make sure that these aren't issues in this PR? I'll continue looking at new framework PRs tomorrow.

@matt-aitken matt-aitken changed the base branch from main to feat/sveltekit-adaptor October 6, 2023 11:52
@matt-aitken matt-aitken merged commit 4cda202 into triggerdotdev:feat/sveltekit-adaptor Oct 6, 2023
matt-aitken added a commit that referenced this pull request Oct 6, 2023
* Feat/svelte kit support (#467)

* feat: implemented svelte-kit adapter

* feat: implemented the @trigger.dev/svelte package

* scaffolded a sveltekit example project with trigger.dev

* added the convertToStandardRequest function in the sveltekit package

* example sveltekit project with the @trigger.dev/svelte package

* doc: added the manual setup guide for sveltekit

* updated the web app onboard for sveltekit

* formatted the manual setup guide for sveltekit

* added a link to webapp sveltekit onboarding

* added a wait function to the sveltekit example app job

* typo fix

* removed comments from the @trigger.dev/svelte

- commented out the useEventRunDetails

* updated package.json to use internal packages

* added a .env.example file

* updated the example-svelte-app

* updated the svelte-example

* updated the onboarding page to reflect requested changes

* made the manual setup guide more specific

* created a .env.example file for svelte-example

* added import aliases

* updated the tsconfig and svelte.config file

* updated to use uint8Array

* added missing paths in the tsconfig

---------

Co-authored-by: Matt Aitken <[email protected]>

* Moved examples to references

* Tweaked the SveleteKit manual setup guide

* Updated zod, and set the svelte/sveltekit package versions to match other packages

* Added the headers to the response from the SvelteKit api/trigger route

* Changeset for Svelte and SvelteKit packages

* Revert change to rawBody

* Remove core from changeset packages

* Delete svelte .npmrc file

* Build svelte package to build, not build/lib

* Made the snapshot instructions easier to copy

* Delete the svelte package

* Latest lockfile

* Updated the SvelteKit docs

* Attempt to get Svelte to import the package

* Updates for port and host options

* Updated lockfile from main

* Update neat-feet-wait.md

---------

Co-authored-by: Chigala <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[TRI-1256] Framework: support for SvelteKit

2 participants