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

feat(jsx-email,app-preview): support tsconfig and jsconfig path aliases in preview #145

Merged
merged 2 commits into from Jan 21, 2024

Conversation

shellscape
Copy link
Owner

@shellscape shellscape commented Jan 21, 2024

Component / Package Name:

This PR contains:

  • bugfix
  • feature
  • refactor
  • documentation
  • other

Are tests included?

  • yes (bugfixes and features will not be merged without tests)
  • no

Breaking Changes?

  • yes (breaking changes will not be merged unless absolutely necessary)
  • no

If yes, please include "BREAKING CHANGES:" in the first commit message body, followed by a description of what is breaking.

List any relevant issue numbers: resolves #129

Description

This PR adds automatic alias support for tsconfig.json (and jsconfig.json) paths https://www.typescriptlang.org/tsconfig#paths in the preview app.

BREAKING CHANGES: This alters default alias paths in the app-preview and jsx-email packages. We'll wait to release this along with some other preview changes.

@shellscape
Copy link
Owner Author

/cc @theDanielJLewis

@shellscape shellscape merged commit 9f48011 into main Jan 21, 2024
2 checks passed
@shellscape shellscape deleted the fix/jsx-email/path-alias branch January 21, 2024 16:36
shellscape added a commit that referenced this pull request Jan 21, 2024
@shellscape
Copy link
Owner Author

Since this was identified as breaking, we're going to revert main and hold off on this until a few more associated changes are made, which are also breaking.

@shellscape shellscape restored the fix/jsx-email/path-alias branch January 22, 2024 01:18
shellscape added a commit that referenced this pull request Jan 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

1 participant