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

feat: use reloadly production proxy #352

Merged
merged 1 commit into from
Oct 25, 2024

Conversation

rndquu
Copy link
Member

@rndquu rndquu commented Oct 25, 2024

This PR uses a proxy URL with whitelisted ip in reloadly dashboard for minting real gift cards.

I've tried to add it to env variable but it followed a whole bunch of refactors because we have to pass the ctx.env object literally to all of the helper methods. I'll address it in a separate PR.

Notes:

  1. This is a temporary solution since we need reloadly gift cards to be working for https://githubuniverse.com/
  2. The web3-gateway-test.com domain DNS is not yet resolved for some of the countries. For example, it doesn't work in Germany and Belgium, but does work in Latvia.

@rndquu rndquu marked this pull request as ready for review October 25, 2024 08:51
@rndquu rndquu requested a review from Keyrxng as a code owner October 25, 2024 08:51
@rndquu rndquu requested a review from EresDev October 25, 2024 08:51
@ubiquity-os-deployer
Copy link

ubiquity-os-deployer bot commented Oct 25, 2024

Copy link
Contributor

github-actions bot commented Oct 25, 2024

Preview Deployment
e5555984e839717f3cf0ae82c39d136c89044bd0

Copy link
Contributor

@EresDev EresDev left a comment

Choose a reason for hiding this comment

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

I have given it a quick run. It is working nicely.

@rndquu rndquu merged commit e555598 into ubiquity:development Oct 25, 2024
3 checks passed
@rndquu rndquu deleted the feat/reloadly-proxy branch October 25, 2024 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants