Skip to content

Conversation

@arv
Copy link
Contributor

@arv arv commented Nov 19, 2024

I thought that this would work now that we have an ICU build. But it doesn't. I'm going to have to do some more work on this.

I thought that this would work now that we have an ICU build. But it
doesn't. I'm going to have to do some more work on this.
@arv arv requested a review from tantaman November 19, 2024 09:45
@vercel
Copy link

vercel bot commented Nov 19, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
replicache-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 19, 2024 10:32am
zbugs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 19, 2024 10:32am

darkgnotic and others added 10 commits November 19, 2024 11:30
now that we can check against `null`, missing params is not an error. A missing param defaults to a `null` value.

To be used in permission code where the `authData` may not exist.
These variables are collections of rules which are policies.
I tried `lit` on for size but it just felt more awkward.

Example:

```ts
issue.where(({cmp, lit}) => cmp(lit(22), '<', 42));

// vs

issue.where(({cmpLit}) => cmpLit(22, '<', 42));

// 1. cmpLit is less to type
// 2. cmpLit doesn't have the confusing asymmetry of only one literal needing to be wrapped in `lit`
```
@tantaman
Copy link
Contributor

I thought that this would work now that we have an ICU build

You got the ICU extensions building?

@arv
Copy link
Contributor Author

arv commented Nov 20, 2024

You got the ICU extensions building?

I thought you got that working :-)

@tantaman
Copy link
Contributor

tantaman commented Nov 20, 2024

I got stuck on the windows build.

@arv
Copy link
Contributor Author

arv commented Aug 27, 2025

This is still broken 😢

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants