Skip to content

feat: (CXSPA-11480) - Use SAP managed repository for dependencies#21282

Draft
Pio-Bar wants to merge 5 commits intodevelopfrom
feature/CXSPA-11480
Draft

feat: (CXSPA-11480) - Use SAP managed repository for dependencies#21282
Pio-Bar wants to merge 5 commits intodevelopfrom
feature/CXSPA-11480

Conversation

@Pio-Bar
Copy link
Copy Markdown
Contributor

@Pio-Bar Pio-Bar commented Mar 25, 2026

Closes: https://jira.tools.sap/browse/CXSPA-11480

To setup SAP_ARTIFACTORY_NPM_TOKEN variable locally, visit https://common.repositories.cloud.sap and login with SSO. Click on the profile and select "set me up". Select NPM, search for npmjs then "Generate Token"

  • Consume packages from common.repositories.cloud.sap/artifactory/api/npm/npmjs. Needs confirmation whether this is the right repository to use.
  • Sets minimumReleaseAge for packages to 3 days. The actual delay time may need to be discussed.

@Pio-Bar Pio-Bar requested a review from a team as a code owner March 25, 2026 12:32
@github-actions github-actions bot marked this pull request as draft March 25, 2026 12:32
@Pio-Bar Pio-Bar marked this pull request as ready for review March 25, 2026 13:06
@cypress
Copy link
Copy Markdown

cypress bot commented Mar 25, 2026

spartacus    Run #52582

Run Properties:  status check failed Failed #52582  •  git commit f887d7288f ℹ️: Merge 8148408510efb54fec80363961838bf7dd00c006 into 5d4c6400b7e57ae2ec38e605dc0c...
Project spartacus
Branch Review feature/CXSPA-11480
Run status status check failed Failed #52582
Run duration 04m 14s
Commit git commit f887d7288f ℹ️: Merge 8148408510efb54fec80363961838bf7dd00c006 into 5d4c6400b7e57ae2ec38e605dc0c...
Committer PioBar
View all properties for this run ↗︎

Test results
Tests that failed  Failures 1
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 100
View all changes introduced in this branch ↗︎

Tests for review

Failed  cypress/e2e/regression/site-context/currency/currency.core-e2e.cy.ts • 1 failed test • B2C

View Output Video

Test Artifacts
Currency change > on the product page > should display the chosen currency Test Replay Video
Flakiness  cypress/e2e/ssr/pages.core-e2e.cy.ts • 3 flaky tests • SSR

View Output Video

Test Artifacts
SSR > should render homepage Test Replay Video
SSR > should render PLP Test Replay Video
SSR > should render PDP Test Replay Video

@github-actions github-actions bot marked this pull request as draft April 2, 2026 19:17
@giancorderoortiz giancorderoortiz marked this pull request as ready for review April 2, 2026 19:22
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

E2E Tests Failed ❌

📸 Download Screenshots


Commit: f887d72

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

Merge Checks Failed

Please push a commit to re-trigger the build.
To push an empty commit you can use `git commit --allow-empty -m "Trigger Build"`

@github-actions github-actions bot marked this pull request as draft April 5, 2026 17:30
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