Skip to content

Conversation

@RyanCavanaugh
Copy link
Member

Reverts #48668

This caused #49307 and we don't have a good mitigation for that break yet. Since the originating bug was not a regression, we'd like a fresh try on that bug which includes a testcase that replicates what's going on with createSlice

@RyanCavanaugh
Copy link
Member Author

@typescript-bot cherry-pick this to release-4.8

@typescript-bot
Copy link
Collaborator

typescript-bot commented Aug 11, 2022

Heya @RyanCavanaugh, I've started to run the task to cherry-pick this into release-4.8 on this PR at eff4406. You can monitor the build here.

@typescript-bot
Copy link
Collaborator

Hey @RyanCavanaugh, I've opened #50280 for you.

typescript-bot pushed a commit to typescript-bot/TypeScript that referenced this pull request Aug 11, 2022
Component commits:
eff4406 Revert "Fixed an issue with contextual type for intersection properties (microsoft#48668)"
This reverts commit 9236e39.
@DanielRosenwasser
Copy link
Member

Just an FYI @Andarist

@RyanCavanaugh RyanCavanaugh merged commit adf26ff into main Aug 12, 2022
@RyanCavanaugh RyanCavanaugh deleted the revert-48668-contextual-type-function-obj-prop-intersection branch August 12, 2022 17:24
amcasey added a commit to amcasey/TypeScript that referenced this pull request Aug 15, 2022
typescript-bot pushed a commit to typescript-bot/TypeScript that referenced this pull request Aug 16, 2022
Component commits:
eff4406 Revert "Fixed an issue with contextual type for intersection properties (microsoft#48668)"
This reverts commit 9236e39.

Co-authored-by: Ryan Cavanaugh <[email protected]>
Andarist added a commit to Andarist/TypeScript that referenced this pull request Jan 4, 2023
@microsoft microsoft locked as resolved and limited conversation to collaborators Oct 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants