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

Update default value for Relay Service URL #2488

Merged

Conversation

fmrsabino
Copy link
Contributor

@fmrsabino fmrsabino commented Sep 4, 2023

What it solves

The -nest name in the domain name should not be used anymore and will be removed in the future. This PR updates the respective value to use the new supported URL.

How this PR fixes it

  • Updates the default value set for SAFE_RELAY_SERVICE_URL_STAGING from https://safe-client-nest.staging.5afe.dev/v1/relay to https://safe-client.staging.5afe.dev/v1/relay
  • Updates the default value set for SAFE_RELAY_SERVICE_URL_PRODUCTION from https://safe-client-nest.safe.global/v1/relay to https://safe-client.safe.global/v1/relay

How to test it

NA

Screenshots

NA

Checklist

  • I've tested the branch on mobile 📱
  • I've documented how it affects the analytics (if at all) 📊
  • I've written a unit/e2e test for it (if applicable) 🧑‍💻

@fmrsabino fmrsabino self-assigned this Sep 4, 2023
@github-actions
Copy link

github-actions bot commented Sep 4, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@fmrsabino fmrsabino requested a review from a team September 4, 2023 15:29
Copy link
Member

@katspaugh katspaugh left a comment

Choose a reason for hiding this comment

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

Thanks!
Should we update prod as well?

@fmrsabino
Copy link
Contributor Author

Thanks! Should we update prod as well?

I was waiting for a confirmation on the production URLs but it seems that they can be updated as well. Do you prefer a separate PR or should I include the update on this one?

@katspaugh
Copy link
Member

This one pls

@fmrsabino fmrsabino force-pushed the update-relay-service-url-staging branch from 9805da6 to 03163bb Compare September 5, 2023 11:19
@fmrsabino fmrsabino changed the title Update default value for Relay Service URL (staging) Update default value for Relay Service URL Sep 5, 2023
Copy link
Member

@katspaugh katspaugh left a comment

Choose a reason for hiding this comment

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

Thank you sir.

@katspaugh katspaugh merged commit b04ba7e into safe-global:dev Sep 5, 2023
3 of 6 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Sep 5, 2023
@fmrsabino fmrsabino deleted the update-relay-service-url-staging branch September 5, 2023 12:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants