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.

Sign up for GitHub

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

Jump to bottom

Update penny to use the app folder #204

Merged
merged 88 commits into from
Mar 21, 2024
Merged

Update penny to use the app folder #204

merged 88 commits into from
Mar 21, 2024

Conversation

mshick
Copy link
Member

@mshick mshick commented Mar 15, 2024 •
edited
Loading

Test Plan (Steps to test):

  1. Basic regression, click around, sign in, check out...
  2. Pull this branch, ensure you can run it locally with npm run dev.
  3. To check out you will need the password test1234 for the store.

@mshick
basic cleanup
29acc7c
@mshick
basic dep bumps
b469873
@mshick
most deps updated, fixing typecheck
18001ce
@mshick
msw and ts at latest reasonable
a8a3c95
@mshick
bump codegen
7b19915
@mshick
sentry configured for app router
b5d2f73
@mshick
use NEXT_PUBLIC var
f629184
@mshick
config cleanup and sentry update
e8b4d70
@mshick
clean up error page
41fc8e1
@mshick
fix up sentry etc
7013bca
@mshick
add worker-src to CSP
859f30a
@mshick
Conforming tsconfig more
b13f82d
@mshick
update all paths, pass typecheck
10d366c
@mshick
convert codegen to ts for better typecheck, fix err
f63afda
@mshick
Merge branch 'sc-12854/feature/update-penny-to-use-next-js-14' into s…
632e258 
…c-12962/feature/update-penny-to-use-the-app-router
@mshick
Fix lint rules and fix
1c7ad7f
@mshick
Final lint removed
4e132bc
@mshick
fix preval issues (temporary)
3016200
@mshick
fix typechecks and lint
847644b
@mshick
fix typecheck and scope it better
e2eafab
@mshick
jest config
c7fe2fd
@mshick
First round migrations
71e0cfb
@mshick
account pages migrated
82b35b7
@mshick
Delete replaced pages
d6dc53f
@mshick
migrating product and collection pages
e87ca67
@mshick
cleanup, remove last pages
22c7ae1
@mshick
remove unused deps
cca6e2d
@mshick
update config and readme
b4a768c
@mshick
remove unused layouts and preval
86ce19c
@mshick
fix type errs
eaaa47f
mshick
mshick commented Mar 19, 2024
View reviewed changes
src/config.ts Outdated Show resolved Hide resolved
mshick
mshick commented Mar 19, 2024
View reviewed changes
src/lib/auth/errors.ts Outdated Show resolved Hide resolved
mshick
mshick commented Mar 19, 2024
View reviewed changes
src/auth.ts Show resolved Hide resolved
mshick
mshick commented Mar 19, 2024
View reviewed changes
.env Show resolved Hide resolved
@mcat
Copy link
Member

mcat commented Mar 20, 2024

Ran into an issue with sending the activation email when a user already exists on a store, but hasn't been activated yet.

Screenshot 2024-03-20 at 09 42 22

@mshick mshick merged commit f527008 into main Mar 21, 2024
18 checks passed
@mshick mshick deleted the sc-12962/feature/update-penny-to-use-the-app-router branch March 21, 2024 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@asprouse asprouse asprouse approved these changes

@mcat mcat Awaiting requested review from mcat

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mshick @mcat @asprouse