-
Notifications
You must be signed in to change notification settings - Fork 6
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
feat: init web app, LEA-2115 #865
base: dev
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #865 +/- ##
=======================================
Coverage 40.06% 40.06%
=======================================
Files 203 203
Lines 9689 9689
Branches 575 575
=======================================
Hits 3882 3882
Misses 5807 5807
|
7cdd37e
to
d6ed2f0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @alter-eggo excited to get this started
apps/web/.eslintrc.cjs
Outdated
@@ -0,0 +1,84 @@ | |||
/** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can this be a esm module like the other packages?
apps/web/app/tailwind.css
Outdated
@@ -0,0 +1,12 @@ | |||
@tailwind base; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Saw your comment on doing this another time, but for removing code, suggest we remove this in this PR, so it never enters the history
apps/web/README.md
Outdated
@@ -0,0 +1,47 @@ | |||
# Welcome to Remix + Cloudflare! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would be nice to commit a basic version of our own readme here - describe the app, present it to the community at large and describe what we are doing here.
@@ -0,0 +1,36 @@ | |||
{ | |||
"include": [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have:
"extends": "../../packages/tsconfig-config/tsconfig.base",
do we want to use that like in mobile
?
d6ed2f0
to
2564465
Compare
Set up of new web app using https://developers.cloudflare.com/pages/framework-guides/deploy-a-remix-site/
will fix configs in sep prs