29 points cpojer | 4 comments | | HN request time: 0.682s | source
1. Reubend ◴[] No.46215623[source]
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.

replies(2): >>46217825 #>>46218692 #
2. cpojer ◴[] No.46217825[source]
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.

3. hombre_fatal ◴[] No.46218692[source]
At least complain about something specific to the project and not generic TypeScript syntax.