Skip to content
New issue

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

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

Already on GitHub? # to your account

feat(view_models): added stackTrace to onError callback for FutureViewModel and StreamViewModel #1149

Conversation

Pebkac03
Copy link
Contributor

@Pebkac03 Pebkac03 commented Jan 4, 2025

Suggestion for adding stackTrace to onError callback in FutureViewModel and StreamViewModel. Fixes #1148.

@Pebkac03 Pebkac03 marked this pull request as ready for review January 4, 2025 07:30
FilledStacks
FilledStacks previously approved these changes Jan 12, 2025
Copy link
Contributor

@FilledStacks FilledStacks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@Pebkac03
Copy link
Contributor Author

all tests passing localy now

@FilledStacks
Copy link
Contributor

The lcov step is failing but all tests are passing so I'll merge this in.

@FilledStacks FilledStacks merged commit 50b214e into Stacked-Org:master Feb 2, 2025
1 check failed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feature]: Add stackTRace to onError override in special ViewModels
2 participants