Skip to content

Replace mentions of demo7.dspace.org and api7.dspace.org with new demo URLs#2452

Merged
hardyoyo merged 1 commit intoDSpace:mainfrom
tdonohue:fix_demo_urls
Aug 22, 2023
Merged

Replace mentions of demo7.dspace.org and api7.dspace.org with new demo URLs#2452
hardyoyo merged 1 commit intoDSpace:mainfrom
tdonohue:fix_demo_urls

Conversation

@tdonohue
Copy link
Member

@tdonohue tdonohue commented Aug 22, 2023

Description

This PR is necessary to finalize the migration from https://demo7.dspace.org to https://demo.dspace.org (and https://sandbox.dspace.org)

It removes all mentions of https://demo7.dspace.org and https://api7.dspace.org from our codebase and configs.

On our main branch, configurations will be updated to use https://sandbox.dspace.org, as this is the new demo site which will automatically sync with the main branch.

On our dspace-7_x branch, configurations will be updated to use https://demo.dspace.org, as this is the new demo site which will automatically sync with the dspace-7_x branch. (This is handled in a separate PR against dspace-7_x, see #2453)

@tdonohue tdonohue added code task 1 APPROVAL pull request only requires a single approval to merge labels Aug 22, 2023
@tdonohue tdonohue added this to the 8.0 milestone Aug 22, 2023
@tdonohue tdonohue self-assigned this Aug 22, 2023
Copy link
Member

@hardyoyo hardyoyo left a comment

Choose a reason for hiding this comment

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

This all looks correct to me. 👍

@tdonohue
Copy link
Member Author

NOTE: I've tested the new configs for https://sandbox.dspace.org/server/ (which already exists) and verified that it is possible to run the UI on http://localhost:4000/ while using the sandbox as a backend. So, this appears to be working.

@hardyoyo hardyoyo merged commit e47586f into DSpace:main Aug 22, 2023
@tdonohue tdonohue deleted the fix_demo_urls branch August 22, 2023 22:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1 APPROVAL pull request only requires a single approval to merge code task

Projects

No open projects
Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants