<@UFT11FKSP> I've created PR (<https://github.com/...
# kvisiont
Tomas Kormanak
05/11/2021, 1:53 PM@Robert Jaros I've created PR (https://github.com/rjaros/kvision/pull/266). It's a draft and I will add more states if you like it.
Tomas Kormanak
05/11/2021, 1:59 PMWe often need to catch 404 exception. It is a valid case that object does not exist. Current implementation does not distinguish between error states and does not expose status code.
r
Robert Jaros
05/11/2021, 2:29 PMNice work! 🙂
t
Tomas Kormanak
05/12/2021, 5:01 PMThanks, I updated MR, ready to merge.
r
Robert Jaros
05/12/2021, 5:30 PMMerged. Thank you!
👍 1
t
Tomas Kormanak
05/13/2021, 2:52 PMI created another one with some improvments.
🙏 1
r
Robert Jaros
05/13/2021, 5:12 PMThe exception is still
ServiceUnaviablet
Tomas Kormanak
05/13/2021, 5:54 PMI've fixed it.
r
Robert Jaros
05/27/2021, 12:16 PMHi. We have unintentionally introduced breaking change with this change 😞 I just have error reports from application in production 🙂
t
Tomas Kormanak
05/27/2021, 12:17 PMHow?
r
Robert Jaros
05/27/2021, 12:17 PMI haven't noticed your
RemoteRequestExceptionThrowableExceptionRobert Jaros
05/27/2021, 12:17 PMso my typical code is failing:
Copy code
try {
restClient......
} catch (e: Exception) {
....
}Robert Jaros
05/27/2021, 12:18 PMCould you make new PR and change
ThrowableExceptiont
Tomas Kormanak
05/27/2021, 12:18 PMOh, sure.
r
Robert Jaros
05/27/2021, 12:19 PMThx!
Robert Jaros
05/27/2021, 12:19 PMI'll make a bugfix release today.
t
Tomas Kormanak
05/27/2021, 12:41 PMAnyway it might be better to catch specific exception than just Exception