Skip to content
This repository has been archived by the owner on Mar 15, 2018. It is now read-only.

don't allow users to rate if cannot rate (bug 1126529/1127388) #992

Merged
merged 2 commits into from
Feb 3, 2015

Conversation

ngokevin
Copy link
Contributor

@ngokevin ngokevin commented Feb 2, 2015

No description provided.

@ngokevin ngokevin changed the title [WIP] don't allow users to rate if cannot rate (bug 1126529/1127388) don't allow users to rate if cannot rate (bug 1126529/1127388) Feb 3, 2015
@ngokevin ngokevin force-pushed the master branch 2 times, most recently from 5ee4463 to 486e5e4 Compare February 3, 2015 19:08
notification.notification({
message: gettext('Sorry, you must purchase this app before reviewing'),
});
});
Copy link
Contributor

Choose a reason for hiding this comment

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

No time param?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nope, just need to bump it ahead of the login notification. I will comment

@@ -12,7 +12,7 @@
{{ _('Edit your review') }}
</a>
</li>
{% else %}
{% elif not (user.logged_in() and not response.user.can_rate) %}
Copy link
Contributor

Choose a reason for hiding this comment

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

Truth table checks out...tricky to read though.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah...saves template though

@spasovski
Copy link
Contributor

Looks good r+

@ngokevin
Copy link
Contributor Author

ngokevin commented Feb 3, 2015

thanks

ngokevin added a commit that referenced this pull request Feb 3, 2015
don't allow users to rate if cannot rate (bug 1126529/1127388)
@ngokevin ngokevin merged commit 1e69e23 into mozilla:master Feb 3, 2015
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.

2 participants