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 intellisense and fix side-effects #10

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Conversation

jerriclynsjohn
Copy link
Collaborator

As we need to use Tier on edge, we had to import the edge-compliant Tier using import {Tier} from 'tier/client', while the entire codebase compiles it renders IDE intellisense useless for fetching Tier SDK documentation and autocomplete.

We fix this by changing tsconfig.json

- "moduleResolution": "Node"
+ "moduleResolution": "NodeNext"

Now that the intellisense is working, there are two side effects that need attention

  1. T3 env validation of schema on build (https://env.t3.gg/docs/nextjs)
  • We cannot import mjs file into a common JS file, so we had to rename env.mjs to env.js, because of this change we cannot import our env file into next.config.mjs and we have to remove it for now.
  1. @headlessui/react - This doesn't break compilation but inherently resurfaces the below mentioned issues

@vercel
Copy link

vercel bot commented Aug 7, 2023

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

Name Status Preview Comments Updated (UTC)
tier-vercel-openai ❌ Failed (Inspect) Aug 10, 2023 9:16pm

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.

1 participant