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

refactor: compare as extension #1069

Merged
merged 9 commits into from
Mar 31, 2022
Merged

Conversation

dhhyi
Copy link
Collaborator

@dhhyi dhhyi commented Mar 15, 2022

PR Type

[x] Refactoring (no functional changes, no API changes)

What Is the Current Behavior?

Compare functionality is already a feature toggle, but not an extension.

What Is the New Behavior?

Refactored to an extension.

Does this PR Introduce a Breaking Change?

[ ] Yes
[ ] No

Other Information

@dhhyi dhhyi added refactoring Refactoring of current code community Community contributions labels Mar 15, 2022
@dhhyi dhhyi self-assigned this Mar 15, 2022
@dhhyi dhhyi force-pushed the refactor/compare-as-extension branch 2 times, most recently from c54f288 to 634b97d Compare March 16, 2022 09:20
@dhhyi dhhyi marked this pull request as ready for review March 16, 2022 10:30
@dhhyi dhhyi removed their assignment Mar 16, 2022
@dhhyi dhhyi force-pushed the refactor/compare-as-extension branch 3 times, most recently from 54cd5ce to 3b14c5b Compare March 25, 2022 12:08
@SGrueber SGrueber self-requested a review March 30, 2022 11:36
@SGrueber SGrueber self-assigned this Mar 30, 2022
@SGrueber SGrueber added this to the 2.2 milestone Mar 30, 2022
@SGrueber SGrueber requested a review from M-Behr March 30, 2022 11:47
@M-Behr M-Behr removed their request for review March 30, 2022 12:01
@dhhyi dhhyi force-pushed the refactor/compare-as-extension branch from 3b14c5b to e92397a Compare March 30, 2022 17:11
SGrueber
SGrueber previously approved these changes Mar 31, 2022
@shauke shauke force-pushed the refactor/compare-as-extension branch from 56fc4be to 0f0f2ca Compare March 31, 2022 19:04
@shauke shauke merged commit 3332337 into develop Mar 31, 2022
@shauke shauke deleted the refactor/compare-as-extension branch March 31, 2022 19:13
SGrueber pushed a commit that referenced this pull request May 19, 2022
BREAKING CHANGES: The compare functionality was moved into an extension (see [Migrations / 2.1 to 2.2](https://github.com/intershop/intershop-pwa/blob/develop/docs/guides/migrations.md#21-to-22) for more details.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Community contributions refactoring Refactoring of current code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants