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

Update contact-summary doc to mention missing report params #964

Merged
merged 1 commit into from Mar 3, 2023
Merged

Update contact-summary doc to mention missing report params #964

merged 1 commit into from Mar 3, 2023

Conversation

jkuester
Copy link
Contributor

As discussed in this forum thread there are a couple of parameters missing in the documentation for the contact-summary.

The appliesIf function for contact-summary cards is called with a report param if there is a report in context.

Also, the modifyContext function is also called with a report param (in addition to the context parameter).

@jkuester jkuester merged commit 73a0584 into main Mar 3, 2023
@jkuester jkuester deleted the fix_contact_summary branch March 3, 2023 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@esthermmoturi esthermmoturi esthermmoturi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@jkuester @esthermmoturi