search

sinclairnick / brail

💌 Transactional email that feels different
https://brail.dev
Apache License 2.0
258 stars 6 forks source link

Migrate to app dir #32

Open sinclairnick opened 1 year ago

FlatMapIO commented 1 year ago

Don't do it, I just jumped out of this mud hole

sinclairnick commented 1 year ago

@FlatMapIO Oh no. What went wrong?

FlatMapIO commented 1 year ago

I have been using appdir for several months. The bugs in appdir will not pop up in the first two days when you just start using it. Once you get into depth, you have to start fighting with all kinds of unpredictable bugs. If you use the edge runtime, the problems will double. Brail is a great project. I suggest brail to stably release under pages and hope it will not have too strong binding relationship with nextjs

jokull commented 1 year ago

The Next.js team have been fixing things as I report them. They'll get there but I agree, not quite there. For what it's worth, I'm still betting on it for my production app launching in about a month - just crossing my fingers 😅.

sinclairnick commented 1 year ago

@FlatMapIO I've been using it in production for a few weeks now (in another project) and while there are certainly a few bugs, it's been manageable.

The core Brail elements have no business being dependent on next or app dir, but the devtools/UI are highly coupled so I'll bear this in mind, considering some people might not want to be forced into using app dir.