actionable error: some errors related to rpc calls are missing debug info #766
Description
Looking at the exceptions raised in python-firestore, I found a couple places where custom ValueErrors are raised after failed rpc calls, discarding all the useful server debugging information
- Transactional.__call__
- Transactional._pre_commit
- AsyncTransactional.__call__
- AsyncTransactional._pre_commit
- Watch._on_snapshot_target_change_remove
These should all use exception chaining to make sure the original error is preserved