Skip to content
This repository has been archived by the owner on Nov 30, 2023. It is now read-only.

move context out of provider file to avoid refresh bug #95

Merged
merged 2 commits into from Oct 4, 2022

Conversation

luthfib
Copy link
Contributor

@luthfib luthfib commented Oct 4, 2022

Moving the CTW Context and provider into separate files to avoid issues when changing certain files it errors out and says context is not defined. This is caused by a bug in vite which there is a current open pr on.. vitejs/vite#10239, which should solve this. Being that this bug is annoying enough in dev i am splitting context and provider into separate files to avoid this bug.

@changeset-bot
Copy link

changeset-bot bot commented Oct 4, 2022

🦋 Changeset detected

Latest commit: 375fdb4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@zus-health/ctw-component-library Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@luthfib luthfib merged commit 112d3df into main Oct 4, 2022
@luthfib luthfib deleted the fix-provider-refresh-bug branch October 4, 2022 16:56
melocule pushed a commit that referenced this pull request Oct 4, 2022
Moving the CTW Context and provider into separate files to avoid issues
when changing certain files it errors out and says context is not
defined. This is caused by a bug in vite which there is a current open
pr on.. vitejs/vite#10239, which should solve
this. Being that this bug is annoying enough in dev i am splitting
context and provider into separate files to avoid this bug.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants