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

Make @umami/state package platform agnostic #2329

Merged
merged 5 commits into from
Jan 21, 2025

Conversation

OKendigelyan
Copy link
Contributor

This PR removes @chakra-ui dependencies from @umami/state and makes it platform agnostic for web and mobile use.

Copy link

vercel bot commented Jan 20, 2025

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

Name Status Preview Comments Updated (UTC)
umami-embed-iframe ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 21, 2025 3:34pm
umami-embed-iframe-ghostnet ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 21, 2025 3:34pm
umami-v2-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 21, 2025 3:34pm
umami-v2-web-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 21, 2025 3:34pm

@OKendigelyan OKendigelyan changed the title Make state package platform agnostic Make @umami/state package platform agnostic Jan 20, 2025
@OKendigelyan OKendigelyan force-pushed the make-state-package-platform-agnostic branch from 24c50df to 8c96a3e Compare January 20, 2025 19:35
@OKendigelyan OKendigelyan force-pushed the make-state-package-platform-agnostic branch from 8c96a3e to d3c72f8 Compare January 20, 2025 19:40
@OKendigelyan OKendigelyan force-pushed the make-state-package-platform-agnostic branch 3 times, most recently from ed7421d to 54dfc39 Compare January 21, 2025 12:14
Copy link
Contributor

@asiia-trilitech asiia-trilitech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @OKendigelyan - the code looks great 🔥 🔥 🔥

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.

2 participants