Skip to content

Add support for multiselect fields in metadata#597

Merged
mattwiller merged 4 commits into
masterfrom
multiselect_metadata
May 4, 2018
Merged

Add support for multiselect fields in metadata#597
mattwiller merged 4 commits into
masterfrom
multiselect_metadata

Conversation

@mattwiller
Copy link
Copy Markdown

Fixes #592

@mattwiller mattwiller requested a review from carycheng May 1, 2018 00:43
@boxcla
Copy link
Copy Markdown

boxcla commented May 1, 2018

Verified that @mattwiller has signed the CLA. Thanks for the pull request!

@santoshaherkar
Copy link
Copy Markdown

When are we planning to make this fix available?

@mattwiller mattwiller merged commit 776ba0b into master May 4, 2018
@mattwiller mattwiller deleted the multiselect_metadata branch May 4, 2018 22:09
@mattwiller
Copy link
Copy Markdown
Author

@santoshaherkar This should go out with our scheduled release next week!

@santoshaherkar
Copy link
Copy Markdown

I don't see support for test operation for multiselect fields. Was that kept out intentionally?

@mattwiller
Copy link
Copy Markdown
Author

@santoshaherkar The SDK should support testing an array element — can you share more details about what you're trying to do that isn't working?

@santoshaherkar
Copy link
Copy Markdown

santoshaherkar commented May 17, 2018

@mattwiller Currently there is no Metadata.test(path, List values) method available to check if multiselect field has given values set.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants