Skip to content

Add anonymous and anoncomments checkboxes#110

Open
rebnwit wants to merge 1 commit into
stiletto:masterfrom
rebnwit:master
Open

Add anonymous and anoncomments checkboxes#110
rebnwit wants to merge 1 commit into
stiletto:masterfrom
rebnwit:master

Conversation

@rebnwit

@rebnwit rebnwit commented Feb 22, 2016

Copy link
Copy Markdown

Как это выглядит:
post w js
post wo js
comment

Comment thread bnw/web/static/dynamic.js Outdated
if (comment_text.val()) {
id += "/" + comment_text.val();
}
var is_anon = anon.is(':checked') ? "anon" : undefined;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why not just use true here instead of "anon"?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Fair enough.

@uzantnomo

Copy link
Copy Markdown
Contributor

LGTM

@stiletto

Copy link
Copy Markdown
Owner

На этой неделе починю конфликты сам и смержу

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants