Skip to content

support SPA dev - #12746

Open
pdurbin wants to merge 5 commits into
developfrom
spa-dev
Open

pdurbin wants to merge 5 commits into
developfrom
spa-dev

Conversation

@pdurbin

@pdurbin pdurbin commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

What this PR does / why we need it:

We want a developer to be able to spin up the backend in this repo and then make use of that running environment while doing frontend development.

Which issue(s) this PR closes:

None. Related:

Special notes for your reviewer:

The bulk of what's going on in this PR is setting up Keycloak enough for the SPA to use it.

Suggestions on how to test this:

Does this PR introduce a user interface change? If mockups are available, please link/include them here:

Is there a release notes update needed for this change?:

Additional documentation:

Comment thread docker-compose-dev.yml
@github-actions

github-actions Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

Test Results

407 tests  +1   393 ✅ +2   23m 50s ⏱️ - 10m 10s
 55 suites ±0    14 💤  - 1 
 55 files   ±0     0 ❌ ±0 

Results for commit e3f0814. ± Comparison against base commit 2553300.

♻️ This comment has been updated with latest results.

From the frontend repo, run this:

npm start -- --host
Note: keycloak.mydomain.com:8090 vs. localhost:8090 should be
explored further. We use the latter in API tests.
@coveralls

coveralls commented Sep 25, 2026 •

Copy link
Copy Markdown

Coverage Status

coverage: 25.411%. remained the same — spa-dev into develop

@pdurbin pdurbin added this to the 6.13 milestone Sep 25, 2026
@pdurbin pdurbin added the Size: 3 A percentage of a sprint. 2.1 hours. label Sep 25, 2026
@sonarqubecloud

Copy link
Copy Markdown

@pdurbin pdurbin removed their assignment Sep 25, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Size: 3 A percentage of a sprint. 2.1 hours.

Projects

Status: Ready for Review ⏩

Development

Successfully merging this pull request may close these issues.

2 participants