top | item 46215623

(no title)

Reubend | 2 months ago

This provides plenty of value conceptually, but I wish they were able to push the syntax into a more intuitive place. My biggest gripe with Relay is how forced the syntax feels, and this seems better but still confusing. Take for example this component declaration:

export const PostCard = ({ post: postRef }: { post: ViewRef<'Post'> }) => { ... }

That feels terrible to me! I don't have any suggestion from my side, but I feel like there's got to be a less awkward way.

discuss

order

cpojer|2 months ago

You don't have to rename the variable. You can also do this:

```

export function PostCard(props: { post: ViewRef<'Post'> }) {

  const post = useView(PostView, props.post);
}

```

If you add a few aliases (that can be easily generated), it becomes this:

```

export function PostCard(props: { post: PostViewRef }) {

  const post = useView(PostView, props.post);
}

```

There are many bits in fate worthy of criticism. Naming and typing React props really isn't one of those.

hombre_fatal|2 months ago

At least complain about something specific to the project and not generic TypeScript syntax.