You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Ross pointed out that our response protos should probably be de-deplicated where possible. I think this would probably be quite helpful, we could write util code that would check any response proto and log it if there was an error.
The text was updated successfully, but these errors were encountered:
The service response messages there can throw away the enum stuff and only include the bits needed. Since they're optional it shouldn't be an issue to have them all combined I think.
Would it make sense to make a ResponseStatus thats included in other responses? Your way seems like it could get quite big, and includes a lot of info from other sources.
Ross pointed out that our response protos should probably be de-deplicated where possible. I think this would probably be quite helpful, we could write util code that would check any response proto and log it if there was an error.
The text was updated successfully, but these errors were encountered: