User Details
- User Since
- Jul 20 2020, 9:28 AM (277 w, 4 d)
- Roles
- Administrator
Thu, Nov 13
Wed, Nov 12
Discussed in 1:1; @kamil will follow-up with responses to my comments later
Tue, Nov 4
Mon, Nov 3
Fri, Oct 31
Thu, Oct 30
This one is hard to review... I'm not really familiar with the .pbxproj format. Regarding the notification service limits, I'm not familiar with those either...
Wed, Oct 29
Tue, Oct 28
Fri, Oct 24
Thu, Oct 23
Makes sense!
D15516 fixes the issue I was referring to.
Here's a failing unit test: https://linear.app/comm/issue/ENG-11509/messages-appear-as-if-from-wrong-user#comment-94575408
We have an object that looks like this:
These buttons not having a loading status was a pet peeve of mine. Thanks for fixing!
I am skeptical of this approach. The core issue is that our FID extraction breaks when unexpected object fields are encountered. What happens if Farcaster introducing a new object field? Won’t the same issue occur? I think we need to fix the core issue instead of reverting this. Let me know if I’m missing something.
Wed, Oct 22
Tue, Oct 21
Mon, Oct 20
We need this for CORS requests
Sun, Oct 19
Sat, Oct 18
Fri, Oct 17
Going to land this as-is, but would appreciate a re-review from @kamil on Monday to make sure the changes address his concern
Good call! I asked Codex to address the concern, and it added an early exit to compaction.upload_files when the user_data is present but the user_keys is missing