Skip to content
This repository was archived by the owner on Jul 1, 2022. It is now read-only.

Use clean(er) Flowdock tags. #33

Merged
merged 1 commit into from
Mar 16, 2015
Merged

Use clean(er) Flowdock tags. #33

merged 1 commit into from
Mar 16, 2015

Conversation

arnested
Copy link
Member

@arnested arnested commented Mar 2, 2015

Flowdock does not accept tags with . in them which a site name usually will do.

This pull request tries to replace some of the "illegal" characters with an underscore.

@arnested
Copy link
Member Author

arnested commented Mar 3, 2015

Hmmm.... dunno why the Drush 6.2.0 test builds stalled / timed out ...

An example of this patch cane be seen at https://www.flowdock.com/app/reload/passagerpulsen/messages/44925 for those @reload with sufficient permissions. The site name was transformed from #tpp.test to #tpp_test.

@xendk
Copy link
Member

xendk commented Mar 3, 2015

Tried restarting it...

xendk added a commit that referenced this pull request Mar 16, 2015
Use clean(er) Flowdock tags.
@xendk xendk merged commit 99b2e63 into master Mar 16, 2015
@xendk xendk deleted the flowdock-tags branch March 16, 2015 09:27
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.

2 participants