Skip to content

[2021-11-04] Client Controlled Nullability - add to list of unanswered questions - null vs error boundary? #798

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
benjie opened this issue Nov 28, 2021 · 3 comments
Assignees
Labels
Action item 🎬 Ready for review 🙌 Action Item issues are reviewed and closed during Working Group meetings.

Comments

@benjie
Copy link
Member

benjie commented Nov 28, 2021

Benjie: I agree that (!) and (?) need to be solved at the same time. Initially (?) made me uncomfortable, and now I understand better why. It could cause issues with your data model to come up that wouldn’t otherwise come up. If the underlying thing would have thrown an error anyway, it probably should still throw an error. These are slightly different behaviors.
Lee: hesitant to have two different things - server-defined vs client-defined error. To take this to first principles - how will people want to use this?
ACTION - add to our list of unanswered questions - it's not necessarily null, more an error boundary?


Note: Action Item issues are reviewed and closed during Working Group
meetings.

@benjie
Copy link
Member Author

benjie commented Nov 28, 2021

@twof Please reply to this so I can assign you. A 👋 will do 👍

@twof
Copy link
Contributor

twof commented Nov 28, 2021

👋

@twof
Copy link
Contributor

twof commented Jan 1, 2022

This has been completed: https://docs.google.com/document/d/10MmZbzfPtk8GoAeSVm_m9ng8Aj10LBXVIQwkpjti6CQ/edit#heading=h.g7wzponnqnef

@benjie benjie added the Ready for review 🙌 Action Item issues are reviewed and closed during Working Group meetings. label Jan 2, 2022
@leebyron leebyron closed this as completed Feb 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Action item 🎬 Ready for review 🙌 Action Item issues are reviewed and closed during Working Group meetings.
Projects
None yet
Development

No branches or pull requests

3 participants