brianRingler / time-distance-tracker

https://time-distance-tracker.vercel.app
0 stars 1 forks source link

modify base url in jsonconfig #2

Closed imaricodes closed 1 year ago

imaricodes commented 1 year ago

To make importing more consistent as project gets more complex. Here's a link on this topic from next.js documentation.

With this change, by default, all imports will reference the /app directory.

Instead of import {someThing} from ' ../../components/someThing.jsx ' , If /components is in the /app directory, you can just write:

import {someThing} from 'components/someThing.jsx'

If we end up with deeper directories, this will come in handy.

One issue with this change is that currently the /models directory is located outside of the app directory. It can still be accessed with the relative path as you have it imported. Not a big deal, it's just that anything outside of the app directory (if this pull request is merged) wouldn't be accessible with the 'shortcut'. Is there some functional reason it should not be in the app directory?

vercel[bot] commented 1 year ago

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

Name Status Preview Comments Updated (UTC)
time-distance-tracker ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 30, 2023 4:27pm