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

Use preferred entity name (Website not website) #21824

Merged
merged 1 commit into from
Oct 13, 2021

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

Use preferred entity name (Website not website) in getfields call

Before

Calls to civicrm_api4('website', 'getfields', []) are failing post caching fixes @colemanw worked on yesterday - with Website being required. I think this should be dealt with in the caching layer but it is ALSO best practice to use ucfirst so I think we should make this change in the meantime. I think this change is probably variable but I'm getting an error trying to send emails via the task

image

After

Working now -

Technical Details

Comments

@civibot
Copy link

civibot bot commented Oct 13, 2021

(Standard links)

@eileenmcnaughton
Copy link
Contributor Author

thanks @seamuslee001

@eileenmcnaughton eileenmcnaughton merged commit bbd9b7b into civicrm:5.43 Oct 13, 2021
@eileenmcnaughton eileenmcnaughton deleted the getfields branch October 13, 2021 22:50
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