Paths

Table of Contentst

[native] improve error handling for TagUnfollowedFarcasterChannel screen
ClosedPublic
Actions

Authored by ginsu on May 6 2024, 12:09 AM.

Details

Reviewers

inka
ashoat

Commits

rCOMM7c234d76cde5: [native] improve error handling for TagUnfollowedFarcasterChannel screen

Summary

This diff improves the error handling for TagUnfollowedFarcasterChannel by providing a few error messages to the user based on the error.message that was given

Depends on D11866

Test Plan

Please see the screenshots below

Screenshot 2024-05-06 at 3.06.48 AM.png (1×1 px, 706 KB)

Screenshot 2024-05-06 at 3.07.18 AM.png (1×1 px, 703 KB)

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ginsu created this revision.May 6 2024, 12:09 AM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptMay 6 2024, 12:09 AM

ginsu edited the summary of this revision. (Show Details)May 6 2024, 12:10 AM

ginsu edited the test plan for this revision. (Show Details)

ginsu added reviewers: inka, ashoat.

ginsu added a parent revision: D11866: [native] implement onPressTagChannel for TagUnfollowedFarcasterChannel.

ginsu attached a referenced file: F1706982: Screenshot 2024-05-06 at 3.07.18 AM.png. (Show Details)

ginsu attached a referenced file: F1706983: Screenshot 2024-05-06 at 3.06.48 AM.png. (Show Details)

ginsu added inline comments.May 6 2024, 12:17 AM

native/community-settings/tag-farcaster-channel/tag-farcaster-channel-by-name.react.js
21	This copy was approved in D11809
22	This copy needs to be reviewed

ginsu added a child revision: D11894: [native] introduce tag channel by name option in action sheet.May 6 2024, 12:22 AM

Harbormaster completed remote builds in B28683: Diff 39827.May 6 2024, 12:26 AM

ginsu requested review of this revision.May 6 2024, 12:26 AM

padding around error message looks different than in invite links, can we make it more consistent?

native/community-settings/tag-farcaster-channel/tag-farcaster-channel-by-name.react.js
21	Can we avoid copy-pasting and have this defined in one place?

address comments

ginsu added inline comments.May 6 2024, 10:35 AM

native/community-settings/tag-farcaster-channel/tag-farcaster-channel-utils.js
5 ↗	(On Diff #39865)	This copy needs to be reviewed

padding around error message looks different than in invite links, can we make it more consistent?

Updated the padding to match the screens of invite links which had 8px of padding above and below the error message

Screenshot 2024-05-06 at 1.36.52 PM.png (1×948 px, 523 KB)

ginsu attached a referenced file: F1709518: Screenshot 2024-05-06 at 1.36.52 PM.png. (Show Details)May 6 2024, 10:38 AM

Harbormaster completed remote builds in B28713: Diff 39865.May 6 2024, 10:51 AM

Just one comment on the copy

native/community-settings/tag-farcaster-channel/tag-farcaster-channel-utils.js
5 ↗	(On Diff #39865)	Let's go with "Could not" instead of "Cannot"

inka accepted this revision.May 7 2024, 5:11 AM

This revision is now accepted and ready to land.May 7 2024, 5:11 AM

address comments

This revision was landed with ongoing or failed builds.May 7 2024, 10:17 AM

Closed by commit rCOMM7c234d76cde5: [native] improve error handling for TagUnfollowedFarcasterChannel screen (authored by ginsu). · Explain Why

This revision was automatically updated to reflect the committed changes.

ginsu added a commit: rCOMM7c234d76cde5: [native] improve error handling for TagUnfollowedFarcasterChannel screen.

Harbormaster completed remote builds in B28740: Diff 39898.May 7 2024, 10:32 AM

Revision Contents
Changeset List

Path

Size

native/

community-settings/

tag-farcaster-channel/

tag-farcaster-channel-by-name.react.js

32 lines

Diff 39827

View Options

native/community-settings/tag-farcaster-channel/tag-farcaster-channel-by-name.react.js

[native] improve error handling for TagUnfollowedFarcasterChannel screenClosedPublicActions