Skip to content
This repository has been archived by the owner. It is now read-only.

feat: Add communications preferences feature toggle. #6

Merged
merged 1 commit into from May 29, 2015
Merged

Conversation

@shane-tomlinson
Copy link
Member

@shane-tomlinson shane-tomlinson commented May 26, 2015

Limit the communications preferences to en users.

Also gives us an easy way to disable communications preferences if needed.

ref:

Limit the communications preferences to `en` users.

Also gives us an easy way to disable communications preferences if needed.
@shane-tomlinson shane-tomlinson force-pushed the communication-prefs branch from 99f59ce to bf6983f May 26, 2015
@rfk
Copy link
Member

@rfk rfk commented May 29, 2015

Is this ready to merge, even if the linked PRs have not landed?

@shane-tomlinson
Copy link
Member Author

@shane-tomlinson shane-tomlinson commented May 29, 2015

Is this ready to merge, even if the linked PRs have not landed?

:shipit:

@shane-tomlinson
Copy link
Member Author

@shane-tomlinson shane-tomlinson commented May 29, 2015

(ship it)

@rfk
Copy link
Member

@rfk rfk commented May 29, 2015

Fortunately github puts the alt tag text on those inline emoticon things, or I'd have no idea what anyone on this team is saying half the time...

rfk added a commit that referenced this pull request May 29, 2015
feat: Add communications preferences feature toggle.
@rfk rfk merged commit 52be31e into master May 29, 2015
@jrgm jrgm deleted the communication-prefs branch May 31, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants