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

[import] Only strengthen references for imported items #1

Closed
rexxars opened this issue Jan 20, 2017 · 2 comments
Closed

[import] Only strengthen references for imported items #1

rexxars opened this issue Jan 20, 2017 · 2 comments

Comments

@rexxars
Copy link
Member

rexxars commented Jan 20, 2017

When running an import with the --missing flag, we are still doing the unsetting of weak reference flags for every document, not just the ones that were missing.

@rexxars rexxars changed the title [core] Only strengthen references for imported items [import] Only strengthen references for imported items Nov 5, 2017
@rexxars
Copy link
Member Author

rexxars commented Nov 5, 2017

@rexxars
Copy link
Member Author

rexxars commented May 15, 2019

Moving this to internal bugtracker

@rexxars rexxars closed this as completed May 15, 2019
rexxars added a commit that referenced this issue Sep 3, 2021
This is considered "the right way", as styled-components generally don't work
well when there are multiple versions on the same page (because of mismatch in
theme context providers, the provided theme values won't be available).

This is step #1 - we'll have to actually make sure styled-components is
installed in each studio (we can't strictly rely on npm7s peer dep behavior).

That task will be handled by the Sanity CLI tool - we'll check on startup,
build, deploy and similar whether or not we have the necessary dependencies
installed, and prompt to install them if not.
rexxars added a commit that referenced this issue Sep 6, 2021
This is considered "the right way", as styled-components generally don't work
well when there are multiple versions on the same page (because of mismatch in
theme context providers, the provided theme values won't be available).

This is step #1 - we'll have to actually make sure styled-components is
installed in each studio (we can't strictly rely on npm7s peer dep behavior).

That task will be handled by the Sanity CLI tool - we'll check on startup,
build, deploy and similar whether or not we have the necessary dependencies
installed, and prompt to install them if not.
rexxars added a commit that referenced this issue Sep 6, 2021
This is considered "the right way", as styled-components generally don't work
well when there are multiple versions on the same page (because of mismatch in
theme context providers, the provided theme values won't be available).

This is step #1 - we'll have to actually make sure styled-components is
installed in each studio (we can't strictly rely on npm7s peer dep behavior).

That task will be handled by the Sanity CLI tool - we'll check on startup,
build, deploy and similar whether or not we have the necessary dependencies
installed, and prompt to install them if not.
rexxars added a commit that referenced this issue Sep 6, 2021
This is considered "the right way", as styled-components generally don't work
well when there are multiple versions on the same page (because of mismatch in
theme context providers, the provided theme values won't be available).

This is step #1 - we'll have to actually make sure styled-components is
installed in each studio (we can't strictly rely on npm7s peer dep behavior).

That task will be handled by the Sanity CLI tool - we'll check on startup,
build, deploy and similar whether or not we have the necessary dependencies
installed, and prompt to install them if not.
ricokahler pushed a commit that referenced this issue Sep 7, 2021
This is considered "the right way", as styled-components generally don't work
well when there are multiple versions on the same page (because of mismatch in
theme context providers, the provided theme values won't be available).

This is step #1 - we'll have to actually make sure styled-components is
installed in each studio (we can't strictly rely on npm7s peer dep behavior).

That task will be handled by the Sanity CLI tool - we'll check on startup,
build, deploy and similar whether or not we have the necessary dependencies
installed, and prompt to install them if not.
rexxars added a commit that referenced this issue Sep 7, 2021
This is considered "the right way", as styled-components generally don't work
well when there are multiple versions on the same page (because of mismatch in
theme context providers, the provided theme values won't be available).

This is step #1 - we'll have to actually make sure styled-components is
installed in each studio (we can't strictly rely on npm7s peer dep behavior).

That task will be handled by the Sanity CLI tool - we'll check on startup,
build, deploy and similar whether or not we have the necessary dependencies
installed, and prompt to install them if not.
rexxars added a commit that referenced this issue Sep 7, 2021
This is considered "the right way", as styled-components generally don't work
well when there are multiple versions on the same page (because of mismatch in
theme context providers, the provided theme values won't be available).

This is step #1 - we'll have to actually make sure styled-components is
installed in each studio (we can't strictly rely on npm7s peer dep behavior).

That task will be handled by the Sanity CLI tool - we'll check on startup,
build, deploy and similar whether or not we have the necessary dependencies
installed, and prompt to install them if not.
rexxars pushed a commit that referenced this issue Jan 26, 2022
Use observables on low-level request, remove eventHandlers from SanityClient
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant