Add support for document_back #1201
Conversation
| @@ -13,10 +13,14 @@ public class StripeLegalEntityVerification : StripeEntity | |||
|
|
|||
| #region Expandable Document | |||
| public string DocumentId { get; set; } | |||
| public string DocumentIdBack { get; set; } | |||
There was a problem hiding this comment.
Minor but I think you should separate each region but will defer to ob
There was a problem hiding this comment.
I had thought we might want to group it into the same region but tbh its been a while since I've worked with c#
There was a problem hiding this comment.
Seems like the purpose is purely an IDE feature. Would you as a developer want to collapse both of these together always or would there be a reason to have this in its own region? Since they're so related I would think we'd want them together but idk.
There was a problem hiding this comment.
The convention so far has been to create a region for each expandable attribute, so I'd have a slight preference for sticking to the convention. That said, it's just an IDE feature so no strong feels here.
There was a problem hiding this comment.
Okay I just placed it in its own region to follow convention. ✅
|
r? @ob-stripe |
|
Alright, pulling this in! |
|
Released as 16.9.0. |
With the introduction of the document_back parameter for both additional owners and the representative we can now update the back of photo IDs 🎉
This PR includes wrappers and tests for the associated wrappers.