Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 1 commit into from
Aug 22, 2023

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
10 checks passed
@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