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

Feature/next app dir mantine v7 #486

Merged
merged 41 commits into from
Sep 20, 2023
Merged

Feature/next app dir mantine v7 #486

merged 41 commits into from
Sep 20, 2023

Conversation

gregrickaby
Copy link
Owner

Related Issue

Fixes #485

Description

  • Upgrade to Mantine v7
  • Upgrade to Next.js 13 App Router

Verification

How will a stakeholder test this PR?

  1. View this on the automated Vercel link below
  2. Verify functionality still works the same as production

@gregrickaby
initial commit after upgrade
3dbd4b8
@gregrickaby
bump dependencies
922f011
@gregrickaby
add lib dir
3ab59e7
@gregrickaby
update metadata and favicon
7c536de
@gregrickaby
check in api/
dd610e7
@gregrickaby
move lib/ to app
0243472
@gregrickaby
update path
eb85311
@gregrickaby
<Meta /> component
b765ccd
@gregrickaby
remove metadata from page
a03d7e4
@gregrickaby
enable dark mode support
f55d4fa
@gregrickaby
initial <Header />
07e082d
@gregrickaby
check-in stylelint and config
3d12379
@gregrickaby
update paths
aa868c0
@gregrickaby
check in <Footer />
b22ac17
@gregrickaby
move to /reddit
f23988d
@gregrickaby
add <BackToTop />
a67eb08
@gregrickaby
add space
d902726
@gregrickaby
check-in <NotFound />
74f1646
@gregrickaby
add <RedditProvider />
fc8f497
@gregrickaby
add docblock
aad75dd
@gregrickaby
rename to <HlsPlayer />
48a126c
@gregrickaby
use reddit provider
394fda1
@gregrickaby
add <Media />
4b5f983
@gregrickaby
add <Results />
302d755
@gregrickaby
add <Search />
9cfff4f
@gregrickaby
add <Settings />
1eeed73
@gregrickaby
add <Sort />
4e13cad
@gregrickaby
style page. import components
db11e72
@gregrickaby
move api routes to old /pages dir
538b205
@gregrickaby
tidy up some styles
cfe6959
@gregrickaby gregrickaby self-assigned this Sep 20, 2023
@vercel
Copy link

vercel bot commented Sep 20, 2023 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
viewer-for-reddit ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 20, 2023 9:52pm

@gregrickaby
Copy link
Owner Author

gregrickaby commented Sep 20, 2023 •
edited
Loading

  • Link colors should be white
  • The footer links (visited) are black, they also need to be white
  • Icons should be white as well

@gregrickaby
Copy link
Owner Author

gregrickaby commented Sep 20, 2023 •
edited
Loading

  • Fix error when adding multiple subs: Error: [@mantine/core] Duplicate options are not supported. Option with value "" was provided more than once

@gregrickaby gregrickaby merged commit 66e39e1 into main Sep 20, 2023
5 checks passed
@gregrickaby gregrickaby deleted the feature/next-app-dir-mantine-v7 branch September 20, 2023 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@gregrickaby gregrickaby

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Next.js App Router + Mantine 7
1 participant
@gregrickaby