Page MenuHomePhabricator

Remove nonfunctional AJAX embed behavior for Slowvote
ClosedPublic

Authored by epriestley on Nov 5 2018, 6:12 PM.
Tags
None
Referenced Files
F15401133: D19773.id47232.diff
Mon, Mar 17, 3:56 PM
F15385111: D19773.id47232.diff
Fri, Mar 14, 9:44 PM
F15382907: D19773.diff
Fri, Mar 14, 2:52 PM
F15368438: D19773.diff
Wed, Mar 12, 2:10 AM
Unknown Object (File)
Sat, Feb 22, 12:45 PM
Unknown Object (File)
Thu, Feb 20, 1:52 AM
Unknown Object (File)
Thu, Feb 20, 1:52 AM
Unknown Object (File)
Thu, Feb 20, 1:52 AM
Subscribers
None

Details

Summary

See https://hackerone.com/reports/434116. Slowvote has a piece of Javascript that attempts to let you vote on {V123} polls inline.

It does not work: nothing ever triggers it (nothing renders a control with a slowvote-option sigil).

At least for now, just remove it. It has a completely separate pathway in the controller and both pathways are buggy, so this makes fixing them easier.

Test Plan

Voted in plurality and approval polls via Slowvote and the embedded widget.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable