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.

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: cookie storage #256

Closed
wants to merge 30 commits into from
Closed

Conversation

juls0730
Copy link

@juls0730 juls0730 commented Apr 24, 2024

Build off of #221 and make SSR better with cookie storage. The original PR seems a bit dead, and I want to get this past the finish line. I'm unsure if manniL or Hossein-Mirazimi forgot about the PR or something, but I'm trying to just get it merged lol

Hossein Mirazimi and others added 29 commits March 17, 2023 19:18
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@juls0730
Copy link
Author

still working on merge conflicts and other small problems, please hold

@juls0730
Copy link
Author

juls0730 commented Apr 24, 2024

that should have the SSR working, and it will look similar to the demo page (whether its supposed to looks broken I'm unsure, but I made it look like the demo page). Once again, I must thank Hossein-Mirazimi and manniL for the original code for all of their great work

@juls0730 juls0730 closed this May 2, 2024
@wkl007
Copy link

wkl007 commented Jun 5, 2024

@juls0730 How is the progress of this pr now, thank you!

@juls0730
Copy link
Author

juls0730 commented Jun 6, 2024

Hey @wkl007 i believe it was working when I closed it, but I can check again and confirm

@juls0730
Copy link
Author

juls0730 commented Jun 6, 2024

update: there is absolutely no way it was working when I closed the PR because I'm pretty confident it was not adding the class server-side, it is now doing that on the latest commit

@wkl007
Copy link

wkl007 commented Jun 6, 2024

@juls0730 Thank you very much for your reply.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants