Change default code organization in skeleton template #1612
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
With #1608 , it will be better to reduce the size of
server.ts
so that we don't need to maintain things that don't change like the cart fragment or the AppSession. Also, I noticed some examples are starting to addutils/xyz
while we already haveutils.ts
in the skeleton template, which would be 1 file and 1 folder with the same name. This PR moves all of that toapp/lib
, following loosely what we have indemo-store
.Thoughts about this?
We should probably update our i18n generator to setup also in
app/lib/i18n.ts
instead ofserver.ts
.