Skip to content
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.

Sign up for GitHub

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

Jump to bottom

ZIO 2 update broke error propagation to the front #37

Open
Esardes opened this issue Oct 7, 2022 · 2 comments
Open

ZIO 2 update broke error propagation to the front #37

Esardes opened this issue Oct 7, 2022 · 2 comments

Comments

@Esardes
Copy link

Esardes commented Oct 7, 2022

FrontendUtils.fetch used to model the error channel and now assumes the call always returns successfully - returning a UIO[A], so that when a backend call fails, the front still sees a success (but will fail to unpickle the result).
@Esardes
Copy link
Author

Esardes commented Oct 10, 2022

@kitlangton if you know which direction you want this to take ("rollback" to previous impl of errors, or "pass an Either to the A, or smthg else) I could work on a PR

@Esardes
Copy link
Author

Esardes commented Nov 2, 2022

@kitlangton I know you're busy so here is me pinging you again to get your thoughts on this

@gkfabs gkfabs mentioned this issue Dec 15, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Esardes