HomePhabricator

Allow a wider range of characters in macro names, including emoji

Description

Allow a wider range of characters in macro names, including emoji

Summary:
Fixes T6121. See PHI357.

  • Allow emoji and other unicode (like Chinese characters) as long as you have at least three of them.
  • Disallow macros with only latin symbols. These were previously allowed.

Test Plan: Created a macro for "๐Ÿถ๐Ÿถ๐Ÿถ", then used it in a comment.

Maniphest Tasks: T6121

Differential Revision: https://secure.phabricator.com/D19051

Details

Provenance
epriestleyAuthored on Feb 9 2018, 10:23 PM
epriestleyPushed on Feb 9 2018, 10:34 PM
Differential Revision
D19051: Allow a wider range of characters in macro names, including emoji
Parents
rP64177cb16e77: Document how webhooks work
Branches
Unknown
Tags
Unknown
Tasks
T6121: Support non-latin characters in macros
Build Status
Buildable 19473
Build 26344: Run Core Tests