Skip to content

Commit 5cacbf6

Browse files
authored
Merge pull request #952 from IQSS/Remove'/SPA'
Replace hard-coded references from /spa to /modern
2 parents a71dff2 + 79ac924 commit 5cacbf6

39 files changed

Lines changed: 262 additions & 175 deletions

File tree

.github/workflows/deploy-beta-testing.yml

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,9 @@ on:
55
branches:
66
- develop
77

8+
env:
9+
FRONTEND_BASE_PATH: /modern
10+
811
jobs:
912
build:
1013
runs-on: ubuntu-latest
@@ -31,7 +34,7 @@ jobs:
3134
run: npm run build
3235

3336
- name: Build with base path
34-
run: npm run build -- --base=/spa
37+
run: npm run build -- --base=${{ env.FRONTEND_BASE_PATH }}
3538

3639
- name: Override runtime config.js for BETA
3740
env:
@@ -115,4 +118,4 @@ jobs:
115118
ASADMIN='/usr/local/payara6/bin/asadmin --user admin'
116119
DATAVERSE_FRONTEND=`$ASADMIN list-applications |grep $APPLICATION_NAME |awk '{print $1}'`
117120
$ASADMIN undeploy $DATAVERSE_FRONTEND
118-
$ASADMIN deploy --name $APPLICATION_NAME --contextroot /spa $APPLICATION_WAR_PATH
121+
$ASADMIN deploy --name $APPLICATION_NAME --contextroot ${{ env.FRONTEND_BASE_PATH }} $APPLICATION_WAR_PATH

.github/workflows/deploy.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ on:
1212
- qa
1313
- demo
1414
basepath:
15-
description: 'Server base path (without slashes) for serving the application (e.g., spa). If left blank, it will try to deploy to the root base path.'
15+
description: 'Server base path (without slashes) for serving the application (e.g., modern). If left blank, it will try to deploy to the root base path.'
1616
type: string
1717
required: false
1818

.github/workflows/generate-war.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ on:
1212
- qa
1313
- demo
1414
basepath:
15-
description: 'Server base path (without slashes) for serving the application (e.g., spa). If left blank, it will try to deploy to the root base path.'
15+
description: 'Server base path (without slashes) for serving the application (e.g., modern). If left blank, it will try to deploy to the root base path.'
1616
type: string
1717
required: false
1818

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,7 @@ This changelog follows the principles of [Keep a Changelog](https://keepachangel
6363
- Upgrade Keycloak to 26.3.2; updated SPI and test realm JSON.
6464
- Truncate long collection and dataset descriptions with expandable content. (#789)
6565
- UI polish: Files Table always shows action buttons. (#800)
66+
- Removed hard-coded references to /spa path and SPA name, changed to /modern (#945)
6667

6768
### Fixed
6869

DEVELOPER_GUIDE.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -236,7 +236,7 @@ project files and shut down any running containers.
236236
237237
Once the script has finished, you will be able to access Dataverse via:
238238
239-
- Dataverse SPA Frontend: [http://localhost:8000/spa][dv_app_localhost_spa_url]
239+
- Dataverse MODERN Frontend: [http://localhost:8000/modern][dv_app_localhost_modern_url]
240240
- Dataverse JSF Application: [http://localhost:8000][dv_app_localhost_legacy_url]
241241
242242
Note: The Dataverse configbaker takes some time to start the application, so the application will not be accessible until
@@ -802,7 +802,7 @@ describe('Create Dataset', () => {
802802
})
803803
804804
it('navigates to the new dataset after submitting a valid form', () => {
805-
cy.visit('/spa/datasets/root/create')
805+
cy.visit(`${FRONTEND_BASE_PATH}/datasets/root/create`)
806806
807807
cy.findByLabelText(/Title/i).type('Test Dataset Title')
808808
cy.findByLabelText(/Author Name/i).type('Test author name', { force: true })
@@ -885,7 +885,7 @@ it(
885885
}
886886
)
887887
888-
cy.visit('/spa/')
888+
cy.visit(`${FRONTEND_BASE_PATH}/`)
889889
890890
// Assertions that rely on your overridden config
891891
cy.findByText('English').should('exist')
@@ -1134,7 +1134,7 @@ path included will redirect to the frontend application.
11341134
[dv_app_localhost_build_url]: http://localhost:5173
11351135
[dv_app_localhost_storybook_url]: http://localhost:6006/
11361136
[dv_app_localhost_designsystem_url]: http://localhost:6007/
1137-
[dv_app_localhost_spa_url]: http://localhost:8000/spa
1137+
[dv_app_localhost_modern_url]: http://localhost:8000/modern
11381138
[dv_app_localhost_legacy_url]: http://localhost:8000/
11391139
11401140
<!-- @gdcc/dataverse -->

README.md

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@
3232
<br />
3333
<br />
3434
<a href="https://www.dataverse.org">Website</a> |
35-
<a href="https://beta.dataverse.org/spa">View Demo (BETA)</a> |
35+
<a href="https://beta.dataverse.org/modern">View Demo (BETA)</a> |
3636
<a href="https://github.com/IQSS/dataverse-frontend/issues">Report Bug</a> |
3737
<a href="https://github.com/IQSS/dataverse-frontend/issues">Request Feature</a>
3838
</p>
@@ -139,21 +139,21 @@ All environments follow an “all-in-one” setup, where the frontend and backen
139139
The **Beta** environment provides a remote space for testing the latest changes. GitHub Actions automatically deploy the current `develop` branches of both the frontend and backend.
140140

141141
- **Audience:** Development team, QA analysts, project managers, selected users for early feedback
142-
- **URL:** [beta.dataverse.org/spa][dv_app_beta_spa_url]
142+
- **URL:** [beta.dataverse.org/modern][dv_app_beta_modern_url]
143143

144144
#### Demo
145145

146146
The **Demo** environment showcases the latest officially released version of the SPA, compatible with the latest Dataverse backend release. Deployments target specific tagged releases (e.g., `0.1.0`) and are performed on demand.
147147

148148
- **Audience:** Project managers, curation team, early adoption testers
149-
- **URL:** [demo.dataverse.org/spa][dv_app_demo_spa_url]
149+
- **URL:** [demo.dataverse.org/modern][dv_app_demo_modern_url]
150150

151151
#### QA
152152

153153
The **QA** environment is a dedicated, short-lived testing space. It is deployed on demand with feature branches (e.g., `feature/xxx`), frequently overwritten, and used for validating new features and bug fixes before merging into development.
154154

155155
- **Audience:** QA analysts, development team
156-
- **URL:** [qa.dataverse.org/spa][dv_app_qa_spa_url]
156+
- **URL:** [qa.dataverse.org/modern][dv_app_qa_modern_url]
157157

158158
#### Spike Environments
159159

@@ -397,15 +397,15 @@ Distributed under the Apache License, Version 2.0. See [LICENSE](LICENSE) for mo
397397
<!-- Application Instances -->
398398
<!-- [dv_app_] -->
399399

400-
[dv_app_beta_spa_url]: https://beta.dataverse.org/spa
401-
[dv_app_demo_spa_url]: https://demo.dataverse.org/spa
402-
[dv_app_qa_spa_url]: https://qa.dataverse.org/spa
400+
[dv_app_beta_modern_url]: https://beta.dataverse.org/modern
401+
[dv_app_demo_modern_url]: https://demo.dataverse.org/modern
402+
[dv_app_qa_modern_url]: https://qa.dataverse.org/modern
403403
[dv_app_beta_legacyjsf_url]: https://beta.dataverse.org
404404
[dv_app_legacyjsf_demo_url]: https://demo.dataverse.org/
405405
[dv_app_localhost_build_url]: http://localhost:5173
406406
[dv_app_localhost_storybook_url]: http://localhost:6006/
407407
[dv_app_localhost_designsystem_url]: http://localhost:6007/
408-
[dv_app_localhost_spa_url]: http://localhost:8000/spa
408+
[dv_app_localhost_modern_url]: http://localhost:8000/modern
409409
[dv_app_localhost_legacy_url]: http://localhost:8000/
410410

411411
<!-- @gdcc/dataverse -->

cypress.config.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ export default defineConfig({
3535
}
3636
},
3737
env: {
38+
frontendBasePath: '/modern',
3839
backendUrl: 'http://localhost:8000',
3940
oidcClientId: 'test',
4041
oidcAuthorizationEndpoint: 'http://localhost:8000/realms/test/protocol/openid-connect/auth',

dev-env/nginx.conf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ http {
104104
}
105105

106106
# Route for SPA frontend
107-
location /spa {
107+
location /modern {
108108
proxy_pass http://frontend:5173;
109109
proxy_http_version 1.1;
110110
proxy_set_header Upgrade $http_upgrade;

dev-env/shib-dev-env/nginx/nginx.conf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@ http {
107107
}
108108

109109
# Route for SPA frontend
110-
location /spa {
110+
location /modern {
111111
proxy_pass http://frontend:5173;
112112
proxy_http_version 1.1;
113113
proxy_set_header Upgrade $http_upgrade;

dev-env/vite.config.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import istanbul from 'vite-plugin-istanbul'
44
import * as path from 'path'
55

66
export default defineConfig({
7+
base: '/modern',
78
plugins: [
89
react(),
910
istanbul({

0 commit comments

Comments
 (0)