Skip to content
This repository has been archived by the owner on May 16, 2019. It is now read-only.

Implement at-pizza #5

Merged
merged 2 commits into from
Apr 19, 2017
Merged

Implement at-pizza #5

merged 2 commits into from
Apr 19, 2017

Conversation

rosylilly
Copy link

I believe @ピザ is a VERY important feature of niconico.

If you feel like it. please merge this.

@binzume
Copy link

binzume commented Apr 19, 2017

👍

@nanocloudx
Copy link

🍕

@ntgaura
Copy link

ntgaura commented Apr 19, 2017

🍕

@@ -77,6 +77,12 @@ export function submitCompose() {
if (!status || !status.length) {
return;
}

if (status.match(/\A\s*[@@]ピザ\s*/) {

Choose a reason for hiding this comment

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

javascript does not support \A in its RegExp, right?

Copy link

Choose a reason for hiding this comment

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

lack of )


if (status.match(/\A\s*[@@]ピザ\s*/) {
window.open('http://info.nicovideo.jp/pizza/', null);
return:

Choose a reason for hiding this comment

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

;

@rosylilly
Copy link
Author

@namusyaka @sorah Thank you for reviewing

@masarakki masarakki merged commit c8d297f into dwango:friends.nico Apr 19, 2017
@masarakki
Copy link

👍

@rosylilly
Copy link
Author

rosylilly commented Apr 19, 2017

Thank you for merge

@suzutan
Copy link

suzutan commented Apr 19, 2017

👍

@rinsuki
Copy link

rinsuki commented Apr 19, 2017

🍕

execjosh pushed a commit to execjosh/mastodon that referenced this pull request May 1, 2017
@rosylilly rosylilly deleted the at-pizza branch August 1, 2017 12:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.