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 t3-env to AvaTax #1636

Merged
merged 10 commits into from
Oct 25, 2024
Merged

Add t3-env to AvaTax #1636

merged 10 commits into from
Oct 25, 2024

Conversation

krzysztofzuraw
Copy link
Member

@krzysztofzuraw krzysztofzuraw commented Oct 25, 2024

Scope of the PR

Adds t3-env library for validating env variables.

I tested:

  • Error flow (with Sentry capture)
  • Working flow (calc taxes)
  • e2e

Related issues

Checklist

Copy link

changeset-bot bot commented Oct 25, 2024

🦋 Changeset detected

Latest commit: 5c4b9d9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
app-avatax Minor

Not sure what this means? Click here to learn what changesets are.

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

Copy link

vercel bot commented Oct 25, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
saleor-app-avatax ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 25, 2024 9:53am
5 Skipped Deployments
Name Status Preview Comments Updated (UTC)
saleor-app-cms ⬜️ Skipped (Inspect) 💬 Add feedback Oct 25, 2024 9:53am
saleor-app-klaviyo ⬜️ Skipped (Inspect) Oct 25, 2024 9:53am
saleor-app-products-feed ⬜️ Skipped (Inspect) 💬 Add feedback Oct 25, 2024 9:53am
saleor-app-search ⬜️ Skipped (Inspect) 💬 Add feedback Oct 25, 2024 9:53am
saleor-app-smtp ⬜️ Skipped (Inspect) 💬 Add feedback Oct 25, 2024 9:53am

@vercel vercel bot temporarily deployed to Preview – saleor-app-klaviyo October 25, 2024 08:50 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-cms October 25, 2024 08:50 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-search October 25, 2024 08:50 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-products-feed October 25, 2024 08:50 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-smtp October 25, 2024 08:50 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-klaviyo October 25, 2024 09:05 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-cms October 25, 2024 09:05 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-products-feed October 25, 2024 09:05 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-smtp October 25, 2024 09:05 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-search October 25, 2024 09:05 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-cms October 25, 2024 09:09 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-smtp October 25, 2024 09:09 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-products-feed October 25, 2024 09:09 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-klaviyo October 25, 2024 09:09 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-search October 25, 2024 09:09 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-cms October 25, 2024 09:32 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-products-feed October 25, 2024 09:32 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-smtp October 25, 2024 09:32 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-products-feed October 25, 2024 09:46 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-klaviyo October 25, 2024 09:46 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-cms October 25, 2024 09:46 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-search October 25, 2024 09:46 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-smtp October 25, 2024 09:46 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-klaviyo October 25, 2024 09:50 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-products-feed October 25, 2024 09:50 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-smtp October 25, 2024 09:50 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-cms October 25, 2024 09:50 Inactive
@vercel vercel bot temporarily deployed to Preview – saleor-app-search October 25, 2024 09:50 Inactive
@krzysztofzuraw krzysztofzuraw marked this pull request as ready for review October 25, 2024 09:59
@krzysztofzuraw krzysztofzuraw requested review from a team as code owners October 25, 2024 09:59
@krzysztofzuraw krzysztofzuraw requested review from a team and Cloud11PL October 25, 2024 09:59
Copy link
Member

Choose a reason for hiding this comment

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

can we import t3 env here, so it validates during build?

Copy link
Member Author

Choose a reason for hiding this comment

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

We can but we will have to install additional pkg: https://env.t3.gg/docs/nextjs#validate-schema-on-build-(recommended) as Next.js don't support importing ts files in config natively

Copy link
Member

Choose a reason for hiding this comment

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

ok, next 15 does so it can wait

@krzysztofzuraw krzysztofzuraw merged commit 7a1a74b into main Oct 25, 2024
18 checks passed
@krzysztofzuraw krzysztofzuraw deleted the add-t3-env-avatax branch October 25, 2024 10:30
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.

4 participants