Skip to content

Commit

Permalink
fix: s3 publishing (#3643)
Browse files Browse the repository at this point in the history
* fix: s3 publishing

* fix: changeset

* Update .changeset/plenty-beds-bake.md

---------

Co-authored-by: Anderson Arboleya <[email protected]>
  • Loading branch information
mchristopher and arboleya authored Jan 29, 2025
1 parent 18bdae8 commit f7f0f0a
Show file tree
Hide file tree
Showing 7 changed files with 18 additions and 15 deletions.
5 changes: 5 additions & 0 deletions .changeset/plenty-beds-bake.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@fuel-ts/account": patch
---

fix: s3 publishing
12 changes: 5 additions & 7 deletions .github/workflows/release.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -166,19 +166,17 @@ jobs:
rootdir: ""
workdir: ""

- uses: aws-actions/configure-aws-credentials@v4
- name: Configure AWS credentials
uses: aws-actions/configure-aws-credentials@v4
if: github.ref_name == 'master' && steps.changesets.outputs.published != 'true'
with:
aws-access-key-id: ${{ secrets.S3_CDN_ACCESS_KEY }}
aws-secret-access-key: ${{ secrets.S3_CDN_SECRET_KEY }}
aws-region: us-east-1
role-to-assume: ${{ env.AWS_ROLE_ARN }}
aws-region: ${{ env.AWS_S3_REGION }}

- name: Upload assets to s3
if: github.ref_name == 'master' && steps.changesets.outputs.published != 'true'
run: |
aws s3 cp ./packages/account/src/providers/assets/images/ s3://${S3_CDN_BUCKET}/assets/ --recursive
env:
S3_CDN_BUCKET: ${{ secrets.S3_CDN_BUCKET }}
aws s3 cp ./packages/account/src/providers/assets/images/ s3://${{ env.AWS_S3_BUCKET }}/providers/ --recursive
# # Commenting out as we require permissions to trigger across repos
# - name: Notify migrations and disclosures of the new release (breaking changes)
Expand Down
2 changes: 1 addition & 1 deletion apps/docs/src/guide/wallets/snippets/connectors.ts
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ class WalletConnector extends FuelConnector {
{
name: 'Ethereum',
symbol: 'ETH',
icon: 'https://cdn.fuel.network/assets/eth.svg',
icon: 'https://assets.fuel.network/providers/eth.svg',
networks: [
{
type: 'ethereum',
Expand Down
2 changes: 1 addition & 1 deletion packages/account/src/providers/assets/assets.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,6 @@ describe('assets', async () => {
});

it.each(assets)('$symbol should have icon resolved to URL', async ({ icon }) => {
expect(icon).toContain('https://cdn.fuel.network/assets');
expect(icon).toContain('https://assets.fuel.network/providers');
});
});
Original file line number Diff line number Diff line change
@@ -1 +1 @@
export const fuelAssetsBaseUrl = 'https://cdn.fuel.network/assets/'
export const fuelAssetsBaseUrl = 'https://assets.fuel.network/providers/'
8 changes: 4 additions & 4 deletions packages/account/src/providers/assets/utils/network.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ describe('Network Utils', () => {
type: 'ethereum',
chainId: CHAIN_IDS.eth.sepolia,
decimals: 18,
icon: 'https://cdn.fuel.network/assets/eth.svg',
icon: 'https://assets.fuel.network/providers/eth.svg',
name: 'Ethereum',
symbol: 'ETH'
})
Expand All @@ -33,7 +33,7 @@ describe('Network Utils', () => {
chainId: CHAIN_IDS.fuel.testnet,
decimals: 9,
assetId: '0xf8f8b6283d7fa5b672b530cbb84fcccb4ff8dc40f8176ef4544ddb1f1952ad07',
icon: 'https://cdn.fuel.network/assets/eth.svg',
icon: 'https://assets.fuel.network/providers/eth.svg',
name: 'Ethereum',
symbol: 'ETH'
})
Expand All @@ -52,7 +52,7 @@ describe('Network Utils', () => {
type: 'ethereum',
chainId: CHAIN_IDS.eth.sepolia,
decimals: 18,
icon: 'https://cdn.fuel.network/assets/eth.svg',
icon: 'https://assets.fuel.network/providers/eth.svg',
name: 'Ethereum',
symbol: 'ETH',
})
Expand All @@ -67,7 +67,7 @@ describe('Network Utils', () => {
chainId: CHAIN_IDS.fuel.testnet,
decimals: 9,
assetId: '0xf8f8b6283d7fa5b672b530cbb84fcccb4ff8dc40f8176ef4544ddb1f1952ad07',
icon: 'https://cdn.fuel.network/assets/eth.svg',
icon: 'https://assets.fuel.network/providers/eth.svg',
name: 'Ethereum',
symbol: 'ETH',
})
Expand Down
2 changes: 1 addition & 1 deletion packages/fuel-gauge/src/e2e-script.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -219,7 +219,7 @@ describe.each(selectedNetworks)('Live Script Test', (selectedNetwork) => {
const expectedBaseAsset = [
{
...expectedRawBaseAsset[0],
icon: 'https://cdn.fuel.network/assets/eth.svg',
icon: 'https://assets.fuel.network/providers/eth.svg',
},
];

Expand Down

0 comments on commit f7f0f0a

Please sign in to comment.