Skip to content

Conversation

@bobo-k2
Copy link
Contributor

@bobo-k2 bobo-k2 commented Mar 20, 2025

A long time ago I added a dApp developer images as base64 encoded data to JSON object. This hurts API (and the portal of course) performance so this PR contains changes to gradually upload base64 data as image to Firebase and to assign the uploaded image link to a dApp developer.

When I say gradually this means that all dApps won't be updated at once (I want to avoid the mayor catastrophe in case of bug). The above mentioned change will happen when

  • a dApp owner updates owned dApp
  • when a single dApp is fetched from API (user access a dApp details from the portal)

@github-actions
Copy link

Visit the preview URL for this PR (updated for commit 66ef159):

https://astar-token-api--pr166-fix-remove-base64-da-lmcromb7.web.app

(expires Thu, 27 Mar 2025 09:55:28 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: f99fa4f4f8f3bb450e6e842f7e1c7783d6d896a3

@bobo-k2 bobo-k2 self-assigned this Mar 20, 2025
@bobo-k2 bobo-k2 requested review from gtg7784 and impelcrypto March 20, 2025 14:35
@bobo-k2 bobo-k2 marked this pull request as ready for review March 20, 2025 14:35
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.

4 participants