Skip to content
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

Add Next.js app dir examples and fix some typing #341

Merged
merged 10 commits into from
Nov 7, 2023

Conversation

jpwilliams
Copy link
Member

@jpwilliams jpwilliams commented Oct 5, 2023

Summary

Also adds two new examples:

  • framework-nextjs-pages-router
  • framework-nextjs-app-router

These examples replace the existing framework-nextjs example, as it's exactly the same as the new pages router.

Checklist

  • Added a docs PR that references this PR N/A
  • Added unit/integration tests
  • Added changesets if applicable N/A
  • Adjust README.md files for examples to be in-line with others

@jpwilliams jpwilliams self-assigned this Oct 5, 2023
@changeset-bot
Copy link

changeset-bot bot commented Oct 5, 2023

⚠️ No Changeset found

Latest commit: 34b542c

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

@jpwilliams jpwilliams changed the base branch from main to bump-next-version October 5, 2023 17:32
Base automatically changed from bump-next-version to main October 5, 2023 17:33
jpwilliams added a commit that referenced this pull request Oct 6, 2023
## Summary
<!-- Succinctly describe your change, providing context, what you've
changed, and why. -->

Fixes #340, ensuring the return type of `"inngest/next"`'s `serve()` is
correctly inferred.

Fuller integration tests for the app router will come in #341.

## Checklist
<!-- Tick these items off as you progress. -->
<!-- If an item isn't applicable, ideally please strikeout the item by
wrapping it in "~~"" and suffix it with "N/A My reason for skipping
this." -->
<!-- e.g. "- [ ] ~~Added tests~~ N/A Only touches docs" -->

- [ ] ~~Added a [docs PR](https://github.com/inngest/website) that
references this PR~~ N/A
- [ ] ~~Added unit/integration tests~~ N/A #341
- [x] Added changesets if applicable

## Related
<!-- A space for any related links, issues, or PRs. -->
<!-- Linear issues are autolinked. -->
<!-- e.g. - INN-123 -->
<!-- GitHub issues/PRs can be linked using shorthand. -->
<!-- e.g. "- inngest/inngest#123" -->
<!-- Feel free to remove this section if there are no applicable related
links.-->
- Fixes #340
Copy link
Contributor

@IGassmann IGassmann left a comment

Choose a reason for hiding this comment

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

Looks good so far

@jpwilliams jpwilliams added the 🧑‍💻 dx Dev-facing resources such as examples or READMEs label Oct 11, 2023
@jpwilliams jpwilliams marked this pull request as ready for review November 7, 2023 12:26
@jpwilliams jpwilliams merged commit 8c428d5 into main Nov 7, 2023
23 checks passed
@jpwilliams jpwilliams deleted the fix/next-app-dir-integration-tests branch November 7, 2023 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧑‍💻 dx Dev-facing resources such as examples or READMEs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants