Paths

Table of Contentst

[native] update FarcasterPrompt based on if the fid is set or not
ClosedPublic
Actions

Authored by ginsu on Mar 29 2024, 2:37 PM.

Details

Reviewers

atul
ashoat

Commits

rCOMMb5530714c2a9: [native] update FarcasterPrompt based on if the fid is set or not

Summary

This diff updates the text content of FarcasterPrompt based on wheter or not the user is logged into farcaster or not (we determine this by whether or not the comm user has an fid set)

Depends on D11483

Test Plan

Please see the screenshots below

When fid is set:

When fid is not set:

Confirmed that these changes to FarcasterPrompt do NOT affect the registration flow

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ginsu created this revision.Mar 29 2024, 2:37 PM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptMar 29 2024, 2:37 PM

Harbormaster completed remote builds in B27832: Diff 38565.Mar 29 2024, 2:52 PM

ginsu requested review of this revision.Mar 29 2024, 2:52 PM

ginsu planned changes to this revision.Mar 29 2024, 3:09 PM

Will this affect the experience when FarcasterPrompt is used from the registration flow?

Will this affect the experience when FarcasterPrompt is used from the registration flow?

No, we set the fid only at the very end of the registration flow so the text content will only change in the context of the Farcaster account settings screen. I will make sure that my test plan shows this

Making @ashoat blocking reviewer since this diff does have copy changes

ginsu added a child revision: D11512: [native] build out UI for farcaster account settings screen.Apr 1 2024, 12:21 PM

Looks fine outside of copy being signed off on

native/components/farcaster-prompt.react.js
14 ↗	(On Diff #38565)	I'd match capitalization and do `fidContext is missing` but up to you

ashoat accepted this revision.Apr 2 2024, 12:26 PM

This revision is now accepted and ready to land.Apr 2 2024, 12:26 PM

address comments

Harbormaster failed remote builds in B27923: Diff 38685!Apr 2 2024, 10:28 PM

rebase before landing + make sure keyserver build ci works

Harbormaster completed remote builds in B27959: Diff 38739.Apr 3 2024, 9:41 AM

Closed by commit rCOMMb5530714c2a9: [native] update FarcasterPrompt based on if the fid is set or not (authored by ginsu). · Explain WhyApr 3 2024, 9:49 AM

This revision was automatically updated to reflect the committed changes.

ginsu added a commit: rCOMMb5530714c2a9: [native] update FarcasterPrompt based on if the fid is set or not.

This diff actually did end up affecting the registration flow. It introduced a very slight regression due to @ginsu's removal of the question mark. @ginsu, can you reply here either with a diff that fixes it, or a task?

ginsu mentioned this in D11621: [native] fix copy in FarcasterPrompt.Apr 10 2024, 9:44 AM

In D11484#333855, @ashoat wrote:

This diff actually did end up affecting the registration flow. It introduced a very slight regression due to @ginsu's removal of the question mark. @ginsu, can you reply here either with a diff that fixes it, or a task?

My bad! thanks for catching this! D11621 should fix this issue

ginsu mentioned this in rCOMMb6331ad353dd: [native] fix copy in FarcasterPrompt.Apr 15 2024, 7:32 PM

Revision Contents
Changeset List

Path

Size

native/

components/

farcaster-prompt.react.js

36 lines

Diff 38739

View Options

native/components/farcaster-prompt.react.js

[native] update FarcasterPrompt based on if the fid is set or notClosedPublicActions