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

Enable APIv4 export action on UFGroup and UFField. #22832

Merged
merged 1 commit into from
Mar 14, 2022

Conversation

jmcclelland
Copy link
Contributor

Overview

With these little changes the export action is available on the UFGroup and UFField Entities.

Before

The export action is not available in APIv4 on the UFField or UFGroup entities.

After

The export action is available.

Technical Details

None.

Comments

It still needs a tweak because UFField exports field_name which hard codes a custom
field's id instead of field_name:name - but I think it's a good first
step.

@civibot
Copy link

civibot bot commented Feb 25, 2022

(Standard links)

@civibot civibot bot added the master label Feb 25, 2022
@colemanw
Copy link
Member

@jmcclelland the style checker wants a blank line before the closing brace.

@jmcclelland
Copy link
Contributor Author

Thanks! Ug - I need to remember to use civilint! Should be better now.

@jmcclelland
Copy link
Contributor Author

Woops - @colemanw - Hm, now I'm not sure why this check failed.

With these little changes the export action is available. It still needs
a tweak because UFField exports `field_name` which hard codes a custom
field's id instead of `field_name:name` - but I think it's a good first
step.
@jmcclelland
Copy link
Contributor Author

@seamuslee001 thanks! I forgot to type git add ... before I typed git commit --amend && git push -f - hopefully this does it.

@colemanw
Copy link
Member

This should be safe to merge now that we're early in the release cycle.

@colemanw colemanw merged commit 4f780f5 into civicrm:master Mar 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants