Closed colbylwilliams closed 1 week ago
👋 Hi! Thank you for this contribution! Just to let you know, our GitHub SDK team does a round of issue and PR reviews twice a week, every Monday and Friday! We have a process in place for prioritizing and responding to your input. Because you are a part of this community please feel free to comment, add to, or pick up any issues/PRs that are labeled with Status: Up for grabs
. You & others like you are the reason all of this works! So thank you & happy coding! 🚀
cc: @JamieMagee
Resolves #520
Before the change?
value_types
and updated thedefault_value
andvalue
properties to allow fornull
,string
, orarray
. This breaks deserialization of payloads with Custom Property or Custom Property Values. See issue for more details.After the change?
multi_select
, andtrue_false
custom property value typesdefault_value
may be null, string, or array of stringsvalue
property may benull
,string
, orarray
of stringsRepository.CustomProperties
to beIDictionary<string, object>
to support polymorphic values of either string or arrayPull request checklist
Does this introduce a breaking change?
Please see our docs on breaking changes to help!