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.

Sign up for GitHub

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

Jump to bottom

APIv4 - Support autocompletes of the Entity entity #25409

Merged
merged 1 commit into from
Jan 26, 2023
Merged

APIv4 - Support autocompletes of the Entity entity #25409

merged 1 commit into from
Jan 26, 2023

Conversation

colemanw
Copy link
Member

@colemanw colemanw commented Jan 23, 2023 •
edited
Loading

Overview

Adds missing metadata to allow browsing API entities with the APIv4 autocomplete select.

Before

  • Insufficient metadata to create autocomplete widget for the APIv4 Entity Entity.
  • Autocomplete for non-dao apis was case-sensitive (incongruous with the majority of apis).

After

Search now works as expected.

@colemanw
APIv4 - Support autocompletes of the Entity entity
c5d8a15 
Allows browsing API entities with the APIv4 autocomplete select
@civibot
Copy link

civibot bot commented Jan 23, 2023

(Standard links)

@civibot civibot bot added the master label Jan 23, 2023
@seamuslee001
Copy link
Contributor

Jenkins re test this please

@colemanw
Copy link
Member Author

Unrelated failure.

FYI @totten - failure was in the upgrade test: "fatal: unable to auto-detect email address (got 'jenkins@rabul-t450s.(none) "

@seamuslee001
Copy link
Contributor

Jenkins re test this please

@seamuslee001 seamuslee001 merged commit dc2db78 into civicrm:master Jan 26, 2023
@seamuslee001 seamuslee001 deleted the entityAutocomplete branch January 26, 2023 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
master merge on pass
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@colemanw @seamuslee001