Skip to content
This repository has been archived by the owner on Jun 23, 2022. It is now read-only.

deps: move to mini-create-react-context #305

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

amankkg
Copy link

@amankkg amankkg commented Jun 14, 2021

Fixes #301

While peer deps were updated in #273, it was not enough actually since create-react-context peerDeps range is not updated. Now it should allow clear installation of React 17

Change is similar to remix-run/react-router#6692 so I believe mini-create-react-context is a drop-in replacement for an old library

I was unable to launch neither project itself or docz locally to test it, but my working project with updated bundles of react-yandex-maps worked well (e.g. YMap, YMaps components)

@happylolonly
Copy link

happylolonly commented Jun 25, 2021

When will you merge? Not working with react 17.0.2 now
@mmarkelov @gribnoysup possible to merge it?

@mmarkelov
Copy link
Contributor

@happylolonly unfortunately i can't merge, because i don't have permissions

@swayok
Copy link

swayok commented Sep 21, 2021

@mmarkelov how about merging? It was approved several months ago...

@mmarkelov
Copy link
Contributor

@mmarkelov how about merging? It was approved several months ago...

I don't have permission to merge

@swayok
Copy link

swayok commented Sep 21, 2021

@mmarkelov can you notify those who can merge?

@gribnoysup
Copy link
Owner

@swayok I get notifications and will look at PRs in detail when I have time.

You're free to fork this code at any moment and republish under a different name or in your private registry, or use a tool like patch-package if you want to address any issues in your application immediately and can't wait

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.

Ошибка при установке react-yandex-maps с 17 версией react
5 participants