Skip to content

refactor: address minor review comments on Standalone Components PR #20793#21130

Merged
Platonn merged 6 commits intodevelopfrom
feature/CXSPA-11889-v1
Feb 24, 2026
Merged

refactor: address minor review comments on Standalone Components PR #20793#21130
Platonn merged 6 commits intodevelopfrom
feature/CXSPA-11889-v1

Conversation

@Platonn
Copy link
Copy Markdown
Contributor

@Platonn Platonn commented Feb 9, 2026

Addresses PR comments from #20793 that were postponed.

For easier review, see individual commits.

fixes https://jira.tools.sap/browse/CXSPA-11889

@Platonn Platonn requested review from a team as code owners February 9, 2026 12:41
@github-actions github-actions Bot marked this pull request as draft February 9, 2026 12:42
pawelfras
pawelfras previously approved these changes Feb 23, 2026
Copy link
Copy Markdown
Contributor

@pawelfras pawelfras left a comment

Choose a reason for hiding this comment

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

Please change the target branch to develop

@Platonn Platonn changed the base branch from release/221121.7.x to develop February 23, 2026 14:19
@Platonn Platonn dismissed pawelfras’s stale review February 23, 2026 14:19

The base branch was changed.

pawelfras
pawelfras previously approved these changes Feb 23, 2026
@Platonn Platonn marked this pull request as ready for review February 24, 2026 11:49
@cypress
Copy link
Copy Markdown

cypress Bot commented Feb 24, 2026

spartacus    Run #51974

Run Properties:  status check passed Passed #51974  •  git commit f8e15f2b31 ℹ️: Merge 5aa3e827e50acc41238dc68d668bb0f8487df443 into 437e5ba62ab0c2297f3a3a3c3eeb...
Project spartacus
Branch Review feature/CXSPA-11889-v1
Run status status check passed Passed #51974
Run duration 11m 02s
Commit git commit f8e15f2b31 ℹ️: Merge 5aa3e827e50acc41238dc68d668bb0f8487df443 into 437e5ba62ab0c2297f3a3a3c3eeb...
Committer Krzysztof Platis
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 3
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 101
⚠️ You've recorded test results over your free plan limit.
Upgrade your plan to view test results.
View all changes introduced in this branch ↗︎

@Platonn Platonn merged commit 02fd4dd into develop Feb 24, 2026
38 checks passed
@Platonn Platonn deleted the feature/CXSPA-11889-v1 branch February 24, 2026 14:28
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.

2 participants