Fate | Not Hacker News!

Discussion (3 comments)

Showing 3 comments

about 1 month ago

2 replies

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.

hombre_fatal

about 1 month ago

It's standard TypeScript syntax of a function that takes an object argument and destructures ones of its keys.

At least complain about something specific to the project.

cpojer

about 1 month 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.

Resources