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

Rewrite #30

Merged
merged 96 commits into from
Jun 18, 2024
Merged

Rewrite #30

merged 96 commits into from
Jun 18, 2024

Conversation

mayank1513
Copy link
Member

Complete rewrite…

We have ensured

  • Minimal API changes

@mayank1513 mayank1513 self-assigned this Jun 17, 2024
deepsource-autofix bot added a commit that referenced this pull request Jun 17, 2024
@deepsource-autofix
style: format code with Prettier
55af94e 
This commit fixes the style issues introduced in 4aca423 according to the output
from Prettier.

Details: #30
@mayank1513 mayank1513 marked this pull request as draft June 17, 2024 19:29
mayank1513 added 17 commits June 18, 2024 00:59
@mayank1513
docs(changeset): Update peer dependencies to include React 19 and Nex…
f1da84a 
…t.js 15
@mayank1513
update peer dependencies
c519f55
@mayank1513
docs(changeset): Fix: Add targetId to ColorSwitch to properly handl…
ab3671c 
…e contenarized themes.
@mayank1513
fix: Add targetId to ColorSwitch to properly handle contenarized th…
7ef208d 
…emes.
@mayank1513
fix: deprecate targetSelector and use targetId
8a20067
@mayank1513
fix: use useStore in place recreating RGS
4f4255c
@mayank1513
refactor: add type specifier for import statement
f787d4b
@mayank1513
fix: fix tests
1222536
@mayank1513
RELEASING: Releasing 6 package(s)
687e140 
Releases:
  nextjs-themes@3.1.3
  app-router@0.0.23
  pages-router@1.0.18
  simple-multi-theme@1.0.18
  tailwind@0.1.12
  vite-example@0.0.23
@mayank1513
fix: fix selectors
c46ac24
@mayank1513
fix: Update downloads link for lite-version
da0c594
@mayank1513
remove cookie from theme-switcher
865b412 
- Will use script injection
@mayank1513
docs(changeset): Remove cookies
8bbf926
@mayank1513
Clean up server side wrapper
7feb476 
- Ensuring no breaking changes.
Kept here only to ensure that existing app do not start throwing errors.
@mayank1513
Do not export internal constants
6a9c9e6
@mayank1513
remove forcedTheme and forcedColorScheme from store as there is no wa…
e6e91a3 
…y to access the store while injecting the scripts
@mayank1513
tmp
d37fbcd
@codecov Codecov
Copy link

codecov bot commented Jun 18, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (d2266dc) to head (6cffb28).

Additional details and impacted files 
@@            Coverage Diff            @@
##              main       #30   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            8         9    +1     
  Lines          406       486   +80     
  Branches        77        64   -13     
=========================================
+ Hits           406       486   +80

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

mayank1513 added 24 commits June 18, 2024 21:13
@mayank1513
introduce forcedState for dynamic forced pages
6354d17
@mayank1513
Add forced-color-scheme
9af4847
@mayank1513
add forced-theme
02acb44
@mayank1513
add forced components
8d47bd4
@mayank1513
dynamic routes
d32880b
@mayank1513
Provide Link component to logo for client side navigation
c0f3e54
@mayank1513
refactor: keep cards list in the shared component
2f69578
@mayank1513
Add router to vite
551a945
@mayank1513
Complete vite example
718e565
@mayank1513
Add linkComponents for examples
64e58c7
@mayank1513
Remove express example
f82bb59 
- Will introduce later with proper express config to serve react app.
@mayank1513
Remove remix example. Remix example is broken and need to be recreated
c55bebc
@mayank1513
updated nextjs app-router
a757976
@mayank1513
touch up pages router examples
cf91d9c
@mayank1513
Add tests for the forced components
1ab70ae
@mayank1513
Update unit tests
3bb7dad
@mayank1513
update lock file
97a43bb
@mayank1513
publish alpha and fix examples build
10e37c8
@mayank1513
rename to app-router
025e864
@mayank1513
Refactor: make more interactive
d08b98d
@mayank1513
RELEASING: Releasing 2 package(s)
e962c7d 
Releases:
  nextjs-themes@4.0.0
  @repo/shared@0.0.1
@mayank1513
downgrade version to 3.1.3 as no breaking changes
189ce41
@mayank1513
update readme
f7deaec
@mayank1513
Add setForced states to useTheme
ff2df36
@mayank1513 mayank1513 marked this pull request as ready for review June 18, 2024 16:17
@mayank1513 mayank1513 merged commit dd88f86 into main Jun 18, 2024
7 of 8 checks passed
@mayank1513 mayank1513 deleted the rewrite branch June 18, 2024 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@mayank1513 mayank1513

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mayank1513