Skip to content

API-125 - Migrated from metaswap to metafi subdomain for proxy and token icons #1060

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

Merged
merged 1 commit into from
Jan 17, 2023

Conversation

blurpesec
Copy link
Contributor

@blurpesec blurpesec commented Jan 11, 2023

PR Title

  • Migrates from *.metaswap.codefi.network to *.metafi.codefi.network for proxy and static subdomains. MetaMask Platform API team is migrating from metaswap aws accounts to their own.

Description

Itemize the changes you have made into the categories below

  • BREAKING:

    • Uses new subdomains for providing the opensea proxy and token icons cache.
  • CHANGED:

    • Uses new subdomains for providing the opensea proxy and token icons cache.
    • New infrastructure enables a new mechanism to handle support requests for changes to token icons. Now https://github.com/MetaMask/contract-metadata will be the priority provider of icons, such that changes there will overwrite any existing token icons we're fetching and caching from other providers.

@blurpesec blurpesec requested a review from a team as a code owner January 11, 2023 18:49
@Gudahtt
Copy link
Member

Gudahtt commented Jan 17, 2023

Looks good! I checked this out locally and found 3 more references to "metaswap.codefi.network":

Have the token-api and api (legacy gas API) services been migrated as well? If so, perhaps these need to be updated.

@evanstucker-hates-2fa
Copy link

api.metaswap.codefi.network was decommissioned. token-api is still located at token-api.metaswap.codefi.network.

@Gudahtt
Copy link
Member

Gudahtt commented Jan 17, 2023

Makes sense! Looks like there is now a conflict, but I can approve once that has been resolved

Copy link
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

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

LGTM!

@Gudahtt Gudahtt merged commit 946edfe into MetaMask:main Jan 17, 2023
MajorLift pushed a commit that referenced this pull request Oct 11, 2023
…ken icons (#1060)

**PR Title**

- Migrates from *.metaswap.codefi.network to *.metafi.codefi.network for
`proxy` and `static` subdomains. MetaMask Platform API team is migrating
from metaswap aws accounts to their own.

**Description**

_Itemize the changes you have made into the categories below_

- BREAKING:

- Uses new subdomains for providing the opensea proxy and token icons
cache.

- CHANGED:

- Uses new subdomains for providing the opensea proxy and token icons
cache.
- New infrastructure enables a new mechanism to handle support requests
for changes to token icons. Now
https://github.com/MetaMask/contract-metadata will be the priority
provider of icons, such that changes there will overwrite any existing
token icons we're fetching and caching from other providers.
MajorLift pushed a commit that referenced this pull request Oct 11, 2023
…ken icons (#1060)

**PR Title**

- Migrates from *.metaswap.codefi.network to *.metafi.codefi.network for
`proxy` and `static` subdomains. MetaMask Platform API team is migrating
from metaswap aws accounts to their own.

**Description**

_Itemize the changes you have made into the categories below_

- BREAKING:

- Uses new subdomains for providing the opensea proxy and token icons
cache.

- CHANGED:

- Uses new subdomains for providing the opensea proxy and token icons
cache.
- New infrastructure enables a new mechanism to handle support requests
for changes to token icons. Now
https://github.com/MetaMask/contract-metadata will be the priority
provider of icons, such that changes there will overwrite any existing
token icons we're fetching and caching from other providers.
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.

3 participants