Skip to content
This repository was archived by the owner on Oct 29, 2020. It is now read-only.

Fixed menu callback, only print picks if they are there#1059

Merged
angaither merged 4 commits intoDoSomethingArchive:devfrom
angaither:staff-pick-form
Mar 10, 2014
Merged

Fixed menu callback, only print picks if they are there#1059
angaither merged 4 commits intoDoSomethingArchive:devfrom
angaither:staff-pick-form

Conversation

@angaither
Copy link
Contributor

Fixes #1050

✈️ 🕐

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo.

@angaither
Copy link
Contributor Author

@aaronschachter updated. no issues adding nid to the var name, I find them easier to read when the variable is at the end of the name.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These guys need the nid prefix added too in order for system settings form to work.

@angaither
Copy link
Contributor Author

@aaronschachter gah, try again!

@aaronschachter
Copy link
Contributor

👍 ship it value added

angaither added a commit that referenced this pull request Mar 10, 2014
Fixed menu callback, only print picks if they are there
@angaither angaither merged commit 20575fe into DoSomethingArchive:dev Mar 10, 2014
@angaither angaither deleted the staff-pick-form branch March 10, 2014 15:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

staff pick campaigns - Mobile Commons id/MailChimp group subscriptions

2 participants