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

Remove / deprecate case param handling #22420

Merged
merged 1 commit into from
Jan 8, 2022
Merged

Remove / deprecate case param handling #22420

merged 1 commit into from
Jan 8, 2022

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

Remove / deprecate case param handling

Before

Handling for parameter that is explictly not passed in - relationshipParams is the $params in this function

image

After

poof

Technical Details

Comments

@eileenmcnaughton
Remove / deprecate case code
ce9a0d0 
It's clear it can't be passed in from the separated off version & it also is not reached from
the importer - which leaves only external usages (not supported but deprecation will do for now)
@civibot
Copy link

civibot bot commented Jan 8, 2022

(Standard links)

@civibot civibot bot added the master label Jan 8, 2022
@demeritcowboy demeritcowboy merged commit fc9af19 into civicrm:master Jan 8, 2022
@eileenmcnaughton eileenmcnaughton deleted the case branch January 9, 2022 01:11
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
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@eileenmcnaughton @demeritcowboy