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

Fix typo in hook that enables links to be added to a block #12913

Merged
merged 1 commit into from
Oct 9, 2018

Conversation

wmortada
Copy link
Contributor

@wmortada wmortada commented Oct 9, 2018

Overview

The code that adds a hook to enable a link to be added to a block has a typo. It says 'shorcuts' rather than 'shortcuts'.

Before

The hook option is 'create.new.shorcuts'.

After

The old hook 'create.new.shorcuts' is still present but deprecated. A new hook 'create.new.shortcuts' has been added.

Technical Details

None. If any extensions use the old option they will still work.

Comments

The documentation refers to the incorrect spelling and needs to be updated. I will do this now.

https://docs.civicrm.org/dev/en/latest/hooks/hook_civicrm_links/

@civibot
Copy link

civibot bot commented Oct 9, 2018

(Standard links)

@eileenmcnaughton
Copy link
Contributor

@colemanw I hope you have a 't' pic for us

@eileenmcnaughton
Copy link
Contributor

fails unrelated

@eileenmcnaughton eileenmcnaughton merged commit b028622 into civicrm:master Oct 9, 2018
@wmortada wmortada deleted the shorcuts branch October 10, 2018 09:09
@colemanw
Copy link
Member

@eileenmcnaughton you mean Mr. T, from our A Team!

image

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.

3 participants