Skip to content

Fix: Revert experimental features (@W-20201209@)#253

Merged
joeluong-sfcc merged 4 commits intomainfrom
ju/revert-subpath-fetch-features
Nov 17, 2025
Merged

Fix: Revert experimental features (@W-20201209@)#253
joeluong-sfcc merged 4 commits intomainfrom
ju/revert-subpath-fetch-features

Conversation

@joeluong-sfcc
Copy link
Contributor

@joeluong-sfcc joeluong-sfcc commented Nov 12, 2025

This PR reverts features that were released in v4.1.0 that are experimental. These new features caused failing tests in PWA Kit, so we're reverting them temporarily and allowing these features to live in a feature branch until we're ready to merge them back in:

#252

* add hot fix

* bump size
@joeluong-sfcc joeluong-sfcc changed the title Release v4.0.1 (#251) Fix: Revert experimental features (@W-20201209@) Nov 12, 2025
@joeluong-sfcc joeluong-sfcc marked this pull request as ready for review November 14, 2025 14:30
@joeluong-sfcc joeluong-sfcc requested a review from a team as a code owner November 14, 2025 14:30
throwOnBadResponse?: boolean;
}

export type FetchFunction = (
Copy link
Contributor

Choose a reason for hiding this comment

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

why do we need to export the FetchFunction if it is not allowed to pass fetch in the config?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We still use this type in environment.ts

Copy link
Contributor

@vcua-mobify vcua-mobify left a comment

Choose a reason for hiding this comment

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

Are we going to release a 4.1.1 with this revert?

@joeluong-sfcc
Copy link
Contributor Author

Are we going to release a 4.1.1 with this revert?

No, we'll be releasing v4.2.0

@joeluong-sfcc joeluong-sfcc merged commit ac1728d into main Nov 17, 2025
6 checks passed
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.

3 participants