Farcaster auth

[daiagi]

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 Integrate farcaster into ProfileV3 #10194

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.

[socket-security]

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package	New capabilities	Transitives	Size	Publisher
npm/@farcaster/auth-client@0.1.1	network	0	105 kB	daddeo

🚮 Removed packages: npm/consola@3.2.3

View full report↗︎

[netlify]

✅ Deploy Preview for koda-canary ready!

Name	Link
🔨 Latest commit	090f256
🔍 Latest deploy log	https://app.netlify.com/sites/koda-canary/deploys/6639229891914f000802ce90
😎 Deploy Preview	https://deploy-preview-10195--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.

[JustLuuuu]

Tried FC data download. I signed the message in the FC app. I think it was successful, but nothing happened on Koda. This is browser after signing:

Screen.Recording.2024-05-03.at.12.19.30.mov

[vikiival]

Sir please have you tested?

1. Login modal will never close (prob bug of that modal?)

I am stuck here

I need to supply both image and banner on my onw

[vikiival]

as per test

[daiagi]

Sir please have you tested?

yes. i did
now will test some more 😄

[daiagi]

@JustLuuuu

you are supposed to scan the QR and the log in via phone

[daiagi]

@vikiival

small yet critical bug

fixed

[daiagi]

one thing

even importing farcaster profile Still Have to upload banner picture

do we want to change that @vikiival @JustLuuuu ?

farcaster + upload banner :
https://deploy-preview-10195--koda-canary.netlify.app/ahp/u/13QX2TXxxRHMrM1CdosnRkpQKTvvdsRFAdpn57jRhAYykJmj

[vikiival]

cc @exezbcz

I would make banner optional for now :)

[vikiival]

you are supposed to scan the QR and the log in via phone

[exezbcz]

I would make banner optional for now :)

it is, if you don't fill it out, it just defaults to the grey.

we can put the small text there

[prury]

this qr code that warpcast provides is unreadable for me haha, had to send the whole link via telegram and access it from mobile, then it worked, not our problem i know.

regarding the whole functionality, agree with banner being optional
and will open another issue for for fields being optional or not.

[vikiival]

If you remove desc the App will crash

[vikiival]

Description is using different component that collection description (a.k.a MD does not work)

Edit does not work.

[vikiival]

But auth works, others can be done in the follow-up

[daiagi]

If you remove desc the App will crash

fixed, by disabling the submit button in edit mode if any required field is missing (name, description, profile pic)

But auth works, others can be done in the follow-up

if so, approve PR please

[codeclimate]

Code Climate has analyzed commit 090f256 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