feat: Replace icons with uploaded profile picture

[Jarsen136]

Thank you for your contribution to the Koda - Generative Art Marketplace.

👇 __ Let's make a quick check before the contribution.

PR Type

• Feature

Needs Design check

• @exezbcz please review

Needs QA check

• @kodadot/qa-guild please review

Context

• Closes Replace icons with uploaded profile picture #10251

Did your issue had any of the "$" label on it?

• My DOT address: Payout

Screenshot 📸

• My fix has changed something on UI; a screenshot is best to understand changes for others.

[netlify]

✅ Deploy Preview for koda-canary ready!

Name	Link
🔨 Latest commit	afde25e
🔍 Latest deploy log	https://app.netlify.com/sites/koda-canary/deploys/663cc1ebd8529f0008e673a9
😎 Deploy Preview	https://deploy-preview-10257--koda-canary.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[prury]

@Jarsen136 from what i could verify, the test is interrupted because the following error is thrown when accessing a non-new-structured profile:

Error: Failing test due to error in browser: FetchError: [GET] "https://profile-beta.kodadot.workers.dev/profiles/5Evgu7n2pVLkj6QfAdDAx11yqmQKstAJbkPCnscYf9pMxTja": 404

is there a way to handle it?

[Jarsen136]

@Jarsen136 from what i could verify, the test is interrupted because the following error is thrown when accessing a non-new-structured profile:

Error: Failing test due to error in browser: FetchError: [GET] "https://profile-beta.kodadot.workers.dev/profiles/5Evgu7n2pVLkj6QfAdDAx11yqmQKstAJbkPCnscYf9pMxTja": 404

is there a way to handle it?

Yup, will fix it

✅ DONE

[prury]

i think we may improve it a bit and not show the old wallet image when opening the profile, maybe a skeleton? @exezbcz wdyt

Gravacao.de.tela.de.07-05-2024.13.47.48.webm

[exezbcz]

amazing! Looking fresh

could you also please change the "address" to the offchain name

i will make some ux improvements followups to ensure it does not compromise the usability

[exezbcz]

@prury hmm interesting that its not in the cache?

[Jarsen136]

amazing! Looking fresh

could you also please change the "address" to the offchain name

✅ Updated

[Jarsen136]

i think we may improve it a bit and not show the old wallet image when opening the profile, maybe a skeleton? @exezbcz wdyt

@prury hmm interesting that its not in the cache?

I have added a better cache for the profile. Could you please test it again?

[prury]

i think we may improve it a bit and not show the old wallet image when opening the profile, maybe a skeleton? @exezbcz wdyt

@prury hmm interesting that its not in the cache?

I have added a better cache for the profile. Could you please test it again?

much better thank you!

[hassnian]

otherwise LGTM

[codeclimate]

Code Climate has analyzed commit afde25e and detected 0 issues on this pull request.

View more on Code Climate.

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud