Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

APIv4 - Support pseudoconstant suffixes in getFields #23348

Merged
merged 1 commit into from
May 3, 2022

Conversation

colemanw
Copy link
Member

@colemanw colemanw commented May 3, 2022

Overview

This builds on the work done in #16154 to allow values in getFields to contain pseudoconstant suffixes.

Before

The APIv4 Explorer allows you to choose values with suffixes... but it doesn't work.

After

Now it does.

@civibot
Copy link

civibot bot commented May 3, 2022

(Standard links)

@civibot civibot bot added the master label May 3, 2022
@eileenmcnaughton
Copy link
Contributor

@colemanw how can I r-run this?

@colemanw
Copy link
Member Author

colemanw commented May 3, 2022

image

@eileenmcnaughton
Copy link
Contributor

Thanks - I was able to replicate this in r-run - merging

@eileenmcnaughton eileenmcnaughton merged commit c0f0578 into civicrm:master May 3, 2022
@eileenmcnaughton eileenmcnaughton deleted the getFieldsSuffixes branch May 3, 2022 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants